Skip to content

Remove test fixture lockfiles, re-enable Dependabot#2096

Merged
KRRT7 merged 7 commits into
mainfrom
ci/dependabot-cleanup
Apr 23, 2026
Merged

Remove test fixture lockfiles, re-enable Dependabot#2096
KRRT7 merged 7 commits into
mainfrom
ci/dependabot-cleanup

Conversation

@KRRT7
Copy link
Copy Markdown
Contributor

@KRRT7 KRRT7 commented Apr 23, 2026

Summary

  • Delete 6 package-lock.json files from code_to_optimize/js/ test fixtures
  • Add code_to_optimize/**/package-lock.json to .gitignore to prevent them from being re-committed
  • Re-enable Dependabot version updates (was disabled with open-pull-requests-limit: 0) with a limit of 5 PRs per ecosystem

Why

24 of our 30 open Dependabot security alerts are from test fixture lockfiles in code_to_optimize/js/. These are not production dependencies — they exist only for JS optimization test scenarios. Removing the lockfiles from git stops the dependency graph from scanning them, which clears the false alerts. The remaining 6 alerts are real Python deps from uv.lock that Dependabot can now create PRs for.

Test plan

  • Verify code_to_optimize/js/ test fixtures still work without lockfiles (npm install regenerates them)
  • After merge, confirm the 24 JS alerts auto-close on GitHub
  • Verify Dependabot starts creating version update PRs within a week

@KRRT7 KRRT7 merged commit bb3a447 into main Apr 23, 2026
11 of 16 checks passed
@KRRT7 KRRT7 deleted the ci/dependabot-cleanup branch April 23, 2026 09:14
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.

1 participant