-
Notifications
You must be signed in to change notification settings - Fork 1
Remove unused files including documentation and update .gitignore for build/test artifacts #112
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Co-authored-by: patrickrb <[email protected]>
patrickrb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot look for files that can be removed in the root of the project too
Co-authored-by: patrickrb <[email protected]>
Found and removed 4 additional unused files from the root directory: Removed files:
Preserved files with verified usage:
Also updated Commit: ffb6aee |
patrickrb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot are the markdown (.md) files actually being used? I want to remove those especially in the root of the project
Co-authored-by: patrickrb <[email protected]>
Removed all 5 markdown documentation files from the root directory: README.md, ADIF_IMPORT_GUIDE.md, INSTALLATION.md, PROPAGATION_FEATURE.md, and DATABASE_SETUP.md. These were standalone documentation files not referenced in the application code. Build still completes successfully with all 79 routes generated. Commit: 189612e |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot leave this file in the project
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Restored README.md file as requested. The file is now back in the project root directory with all its original content.
Commit: 3176dac
Co-authored-by: patrickrb <[email protected]>
patrickrb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested the deployed env, LGTM..merging
This PR cleans up unused development artifacts, test files, and documentation from the repository while carefully preserving all files that are actually used in the codebase or installation processes.
Files Removed (28 total)
From root directory:
ADIF_IMPORT_GUIDE.md,INSTALLATION.md,PROPAGATION_FEATURE.md,DATABASE_SETUP.mdtest-install-check.js,test-propagation.sh.next.bak/directory,test-results/directoryFrom scripts/ directory:
scripts/docker-clean.sh- Docker cleanup utility with no references in codebasescripts/docker-dev.sh- Docker development utility with no references in codebaseFrom test directories:
playwright-report/directory (18 files) - Auto-generated test reports and tracesFiles Preserved
Root directory:
README.md- Main project documentation (restored as requested)vercel.json- Vercel deployment configurationpostgres-lotw-migration.sql- Referenced in Docker/installation documentationpropagation-schema.sql- Referenced in feature documentation.env.docker- Docker environment configuration.gitignore Updated
Added entries to prevent future commits of build and test artifacts:
Analysis Process
Conducted thorough analysis to ensure only truly unused files were removed:
Verification
Both linting and building pass without issues:
npm run build # ✅ Compiled successfully, all 79 routes generatedThe cleanup removes documentation files and development artifacts while preserving complete functionality of the Nextlog amateur radio logging application, including the main project README.
Fixes #111.
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.