Skip to content

Repo hygiene: remove package-lock.json, harden Actions, add Dependabot cooldown#92

Merged
coliff merged 3 commits into
masterfrom
copilot/remove-package-lock-json
Mar 6, 2026
Merged

Repo hygiene: remove package-lock.json, harden Actions, add Dependabot cooldown#92
coliff merged 3 commits into
masterfrom
copilot/remove-package-lock-json

Conversation

Copilot AI commented Mar 6, 2026

Copy link
Copy Markdown
Contributor

Addresses several maintenance and security hygiene items: remove package-lock.json from tracking, update GitHub Actions versions, enforce credential safety on checkout, and throttle Dependabot update frequency.

Changes

  • .gitignore — Added package-lock.json to prevent re-commit
  • package-lock.json — Deleted from repo
  • codeql-analysis.yml / ci.yml — Updated actions/checkout to v4; added persist-credentials: false to both checkout steps to avoid persisting credentials in the local git config
  • dependabot.yml — Added cooldown: default-days: 7 to both npm and github-actions ecosystems to introduce a 7-day delay before Dependabot opens update PRs
Original prompt

This section details on the original issue you should resolve

<issue_title>Updates</issue_title>
<issue_description>lets make the following changes

  • Remove package-lock.json
  • update codeql-analysis github actions to latest versions
  • actions/checkout should have persist-credentials: false set.
  • Dependabot Dependencies should have a cooldown period of 7 days</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: coliff <1212885+coliff@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove package-lock.json and update GitHub actions Repo hygiene: remove package-lock.json, harden Actions, add Dependabot cooldown Mar 6, 2026
@coliff coliff marked this pull request as ready for review March 6, 2026 08:41
Copilot AI review requested due to automatic review settings March 6, 2026 08:41
@coliff coliff merged commit d17fd15 into master Mar 6, 2026
5 of 30 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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@v4 and disable credential persistence via persist-credentials: false.
  • Remove package-lock.json from the repo.
  • Add a 7-day Dependabot cooldown for both npm and github-actions ecosystems.

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.

Comment thread .github/workflows/ci.yml
Comment on lines +28 to 32
uses: actions/checkout@v4
with:
persist-credentials: false

- name: ⎔ Setup node ${{ matrix.node }}

Copilot AI Mar 6, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copilot uses AI. Check for mistakes.
@coliff coliff deleted the copilot/remove-package-lock-json branch March 6, 2026 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Updates

3 participants