fix(repo): replace remaining instances of yarn with pnpm #16571
Conversation
meeroslav
requested review from
FrozenPandaz,
vsavkin,
StalkAltan,
JamesHenry,
nartc,
jaysoo,
ndcunningham,
xiongemi and
AgentEnder
as code owners
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 2572b0a. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this branch ✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
meeroslav
force-pushed
the
chore-fix-yarn-leftovers
branch
2 times, most recently
from
4a8d2f3 to
7673b6f
Compare
meeroslav
force-pushed
the
chore-fix-yarn-leftovers
branch
2 times, most recently
from
1570e7c to
46c6be6
Compare
meeroslav
force-pushed
the
chore-fix-yarn-leftovers
branch
from
46c6be6 to
2572b0a
Compare
| @@ -102,7 +102,7 @@ | |||
| }, | |||
| "generateLockfile": { | |||
| "type": "boolean", | |||
| "description": "Generate a lockfile (e.g. yarn.lock) that matches the workspace lockfile to ensure package versions match.", | |||
| "description": "Generate a lockfile (e.g. package-lock.json) that matches the workspace lockfile to ensure package versions match.", | |||
There was a problem hiding this comment.
I think it's fine but is there a reason we replaced the description in the schemas as well?
There was a problem hiding this comment.
The npm package manager is far more used, so it makes sense to have that as a default example.
Doesn't break anything, just our docs are full of references to yarn despite it not being the most common choice.
|
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Current Behavior
Expected Behavior
Related Issue(s)
Fixes #