Skip to content

pin dependencies#75

Open
munrocket wants to merge 1 commit intotheQRL:mainfrom
munrocket:pin
Open

pin dependencies#75
munrocket wants to merge 1 commit intotheQRL:mainfrom
munrocket:pin

Conversation

@munrocket
Copy link

@munrocket munrocket commented Feb 4, 2026

We all tired of npm supply chain attacks. Let's pin dependencies and update it with npx npm-check-updates -u.

Refs:
https://unit42.paloaltonetworks.com/npm-supply-chain-attack/
https://trustwallet.com/blog/announcements/trust-wallet-browser-extension-v268-incident-community-update

Copy link
Contributor

@jplomas jplomas left a comment

Choose a reason for hiding this comment

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

This is the monorepo package.json

Dependencies are also per-package (see packages/ folder)

@jplomas
Copy link
Contributor

jplomas commented Feb 4, 2026

Thank you for your PR. While this has been roadmapped for final release, given the maturity of this codebase I have no issues with this defence-in-depth once changes made (this PR only touches the monorepo devDependencies, not the package dependencies or devDepenedencies).

Feel free to update PR with changes as suggested.

Copy link
Contributor

@jplomas jplomas left a comment

Choose a reason for hiding this comment

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

Lockfile will also need committing

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