-
Notifications
You must be signed in to change notification settings - Fork 173
feat: migrate to pnpm #2922
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
feat: migrate to pnpm #2922
Conversation
|
|
✅ PR title follows Conventional Commits specification. |
|
Semgrep found 10
Risk: Affected versions of @babel/traverse, babel-traverse, @babel/plugin-transform-runtime, @babel/preset-env, @babel/helper-define-polyfill-provider, babel-plugin-polyfill-corejs2, babel-plugin-polyfill-corejs3, babel-plugin-polyfill-es-shims, and babel-plugin-polyfill-regenerator are vulnerable to Incomplete List Of Disallowed Inputs. An attacker can exploit a vulnerability in the internal Babel methods Manual Review Advice: A vulnerability from this advisory is reachable if you use Babel to compile untrusted JavaScript Fix: Upgrade this library to at least version 0.5.3 at blade/packages/blade/pnpm-lock.yaml:5190. Reference(s): GHSA-67hx-6x53-jw92, CVE-2023-45133 🔴 Fix or ignore this finding to merge your pull request. |
|
Semgrep found 10
Risk: Affected versions of @babel/traverse, babel-traverse, @babel/plugin-transform-runtime, @babel/preset-env, @babel/helper-define-polyfill-provider, babel-plugin-polyfill-corejs2, babel-plugin-polyfill-corejs3, babel-plugin-polyfill-es-shims, and babel-plugin-polyfill-regenerator are vulnerable to Incomplete List Of Disallowed Inputs. An attacker can exploit a vulnerability in the internal Babel methods Manual Review Advice: A vulnerability from this advisory is reachable if you use Babel to compile untrusted JavaScript Fix: Upgrade this library to at least version 0.5.3 at blade/pnpm-lock.yaml:6437. Reference(s): GHSA-67hx-6x53-jw92, CVE-2023-45133 🔴 Fix or ignore this finding to merge your pull request. |
🛡️ Coverage ReportSummaryFull Coverage Details |
| }; | ||
| }); | ||
|
|
||
| const StyledActionList = ({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is there code changes?
Description
so jest uses canvas. we have added that to dev. deps. somehow pnpm as not able to find it.
more update -
https://medium.com/@piotrkorowicki/pnpm-overrides-eb16db96e763
pnpm/pnpm#4920 (comment)
i have documented the migration process here -
https://razorpay.slack.com/archives/C01H13RTF8V/p1756444768657309
Changes
Additional Information
Component Checklist