Summary
public/js/activity.js, public/css/app.css, public/css/signin.css, and public/mix-manifest.json are currently committed to the repository. This adds git noise, risks source/compiled drift, and makes PRs harder to review.
Implementation
1. Update .gitignore — un-comment the three lines that are currently commented out:
/public/js/
/public/css/
/public/mix-manifest.json
2. Remove the committed files from git tracking:
git rm --cached public/js/activity.js public/js/dashboard.js
git rm --cached public/css/app.css public/css/signin.css
git rm --cached public/mix-manifest.json
3. Update README.md — add a build step to the installation instructions:
4. (Optional) Add a CI step that runs npm run prod before deployment so the built assets are always fresh.
Acceptance Criteria
Summary
public/js/activity.js,public/css/app.css,public/css/signin.css, andpublic/mix-manifest.jsonare currently committed to the repository. This adds git noise, risks source/compiled drift, and makes PRs harder to review.Implementation
1. Update
.gitignore— un-comment the three lines that are currently commented out:2. Remove the committed files from git tracking:
3. Update
README.md— add a build step to the installation instructions:4. (Optional) Add a CI step that runs
npm run prodbefore deployment so the built assets are always fresh.Acceptance Criteria
git statusafter building