Repo hygiene: remove package-lock.json, harden Actions, add Dependabot cooldown#92
Conversation
Co-authored-by: coliff <1212885+coliff@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR focuses on repository maintenance/security hygiene by removing the npm lockfile from version control and tightening GitHub Actions + Dependabot configuration.
Changes:
- Update GitHub Actions workflows to use
actions/checkout@v4and disable credential persistence viapersist-credentials: false. - Remove
package-lock.jsonfrom the repo. - Add a 7-day Dependabot cooldown for both
npmandgithub-actionsecosystems.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
.github/workflows/codeql-analysis.yml |
Bumps checkout action to v4 and disables persisted credentials for the CodeQL workflow. |
.github/workflows/ci.yml |
Bumps checkout action to v4 and disables persisted credentials for CI. |
.github/dependabot.yml |
Adds a 7-day cooldown to Dependabot update configuration. |
package-lock.json |
Removes the committed npm lockfile from the repository. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| uses: actions/checkout@v4 | ||
| with: | ||
| persist-credentials: false | ||
|
|
||
| - name: ⎔ Setup node ${{ matrix.node }} |
There was a problem hiding this comment.
package-lock.json is being removed/ignored, but this workflow still uses actions/setup-node with cache: npm and runs npm ci. Both typically require a committed lockfile; without it the job is likely to fail. Either keep package-lock.json tracked, or switch CI to npm install and remove/adjust npm caching accordingly.
Addresses several maintenance and security hygiene items: remove
package-lock.jsonfrom tracking, update GitHub Actions versions, enforce credential safety on checkout, and throttle Dependabot update frequency.Changes
.gitignore— Addedpackage-lock.jsonto prevent re-commitpackage-lock.json— Deleted from repocodeql-analysis.yml/ci.yml— Updatedactions/checkouttov4; addedpersist-credentials: falseto both checkout steps to avoid persisting credentials in the local git configdependabot.yml— Addedcooldown: default-days: 7to bothnpmandgithub-actionsecosystems to introduce a 7-day delay before Dependabot opens update PRsOriginal prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.