-
-
Notifications
You must be signed in to change notification settings - Fork 842
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
chore: add packageManager
field
#389
Conversation
to pin pnpm@7
|
@sxzz is attempting to deploy a commit to the wagmi Team on Vercel. A member of the Team first needs to authorize it. |
Just wondering... Should we pin this to 8.3.1 already? I find myself constantly switching between v7 and v8 to not create unrelated lock file changes when working on different OSS projects. Would be great to bump this to v8 imho. |
@fubhy Done |
New dependency changes detected. Learn more about Socket for GitHub ↗︎ 👍 No new dependency issues detected in pull request Bot CommandsTo ignore an alert, reply with a comment starting with Pull request alert summary
📊 Modified Dependency Overview:
|
Great. Looks good imho. |
971d1d7
to
26197d0
Compare
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.
Would be nice to make sure pnpm versions match in the CI workflows via matrix.pnpm-version
.
Hopefully we can just use Corepack with the workflows some day.
@tmm Done. |
5ffe6ad
to
27aacc2
Compare
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
27aacc2
to
30853f2
Compare
to pin
pnpm@8
PR-Codex overview