Conversation
Pnpm 11 will move most configuration from `.npmrc` to `pnpm-workspace.yaml`. We may as well get a head start on that. Then let's make some changes too: * save-exact - We stopped having Renovate pin in #27087. May as well have this match. * minimumReleaseAge - New setting in 10.16, intended to help avoid installing compromised packages by waiting a day before upgrading. * trustPolicy - New setting in 10.21, intended to help avoid installing compromised packages by rejecting installation when the new version has no provenance and an older version does. * trustPolicyExclude - New setting in 10.22, to override trustPolicy when there's a legitimate reason. This also takes the opportunity to clean up a few things: * public-hoist-pattern was set to the default value. * resolution-mode was only changed from 8.0.0–8.6.12. The current value is the default since then. * use-lockfile-v6 is obsolete. * Versions of `swiper` we use no longer have a build script.
|
Are you an Automattician? Please test your changes on all WordPress.com environments to help mitigate accidental explosions.
Interested in more tips and information?
|
|
Thank you for your PR! When contributing to Jetpack, we have a few suggestions that can help us test and review your patch:
This comment will be updated as you work on your PR and make changes. If you think that some of those checks are not needed for your PR, please explain why you think so. Thanks for cooperation 🤖 Follow this PR Review Process:
If you have questions about anything, reach out in #jetpack-developers for guidance! |
Code Coverage SummaryThis PR did not change code coverage! That could be good or bad, depending on the situation. Everything covered before, and still is? Great! Nothing was covered before? Not so great. 🤷 |
…e of @storybook/test-runner stops using
tbradsha
left a comment
There was a problem hiding this comment.
Artifacts look unchanged and CI is happy. A few inline questions for my own education.
| # Silence this warning. Our `jetpack dependencies build-order` (also used by `jetpack build --all`) checks for cycles itself, plus it has a way to indicate that a dep is only for testing. | ||
| ignoreWorkspaceCycles: true | ||
|
|
||
| # Package compromise avoidance: let new releases age a day, and don't allow trust downgrades unless explicitly excepted. |
There was a problem hiding this comment.
How will this work with Calypso packages? Does this just mean we need to explicitly state the update version?
There was a problem hiding this comment.
We can set minimumReleaseAgeExclude for them if we want.
I haven't tested, but my guess is that trying to set the minimum version to a less-than-1440-minute old dep will make pnpm raise an error.
There was a problem hiding this comment.
Yeah, I wonder if we should do this:
minimumReleaseAgeExclude:
- '@automattic/*'
There was a problem hiding this comment.
May as well do it preemptively, I suppose.
There was a problem hiding this comment.
Added a couple more:
Closes MONOREP-233
Proposed changes:
Pnpm 11 will move most configuration from
.npmrctopnpm-workspace.yaml. We may as well get a head start on that.Then let's make some changes too:
This also takes the opportunity to clean up a few things:
swiperwe use no longer have a build script.Other information:
Jetpack product discussion
pdWQjU-1tL-p2
Does this pull request change what data or activity we track or use?
No
Testing instructions: