Skip to content

chore: remove package-lock.json#1091

Merged
yogeshchoudhary147 merged 1 commit intomainfrom
chore/remove-package-lock
Apr 15, 2026
Merged

chore: remove package-lock.json#1091
yogeshchoudhary147 merged 1 commit intomainfrom
chore/remove-package-lock

Conversation

@yogeshchoudhary147
Copy link
Copy Markdown
Contributor

@yogeshchoudhary147 yogeshchoudhary147 commented Apr 15, 2026

Summary

  • Remove package-lock.json and add it to .gitignore to better support alternative registries
  • Replace npm ci with npm i in all CI workflows (build, test, integration, publish actions)
  • Remove cache: npm from all setup-node steps and replace with package-manager-cache: false
  • Remove registry-url from npm-publish action (not needed with OIDC publishing)
  • Add explicit npm i step to snyk.yml (previously relied on the lockfile to trigger its own install)

@yogeshchoudhary147 yogeshchoudhary147 requested a review from a team as a code owner April 15, 2026 12:59
@yogeshchoudhary147 yogeshchoudhary147 force-pushed the chore/remove-package-lock branch 3 times, most recently from 4f3c756 to c4ea2ab Compare April 15, 2026 13:06
- Remove package-lock.json and add to .gitignore
- Replace npm ci with npm i in all CI workflows
- Remove cache: npm from setup-node steps
@yogeshchoudhary147 yogeshchoudhary147 force-pushed the chore/remove-package-lock branch from c4ea2ab to bccd14f Compare April 15, 2026 13:19
@yogeshchoudhary147 yogeshchoudhary147 merged commit 6a94678 into main Apr 15, 2026
12 checks passed
@yogeshchoudhary147 yogeshchoudhary147 deleted the chore/remove-package-lock branch April 15, 2026 13:33
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.

2 participants