Skip to content
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

Ensure yarn is used for next-with-deps #44666

Merged
merged 1 commit into from
Jan 6, 2023
Merged

Conversation

ijjk
Copy link
Member

@ijjk ijjk commented Jan 6, 2023

We aren't able to use pnpm for this script as it will no-op inside of the Next.js repo so we need to use a different package manager instead.

x-ref: slack thread

Verified

This commit was signed with the committer’s verified signature.
ijjk JJ Kasper
@ijjk ijjk added the created-by: Next.js team PRs by the Next.js team. label Jan 6, 2023
@ijjk ijjk requested a review from wbinnssmith January 6, 2023 21:38
@ijjk ijjk merged commit 93f4a1c into vercel:canary Jan 6, 2023
@ijjk ijjk deleted the fix/next-with-deps branch January 6, 2023 21:43
kodiakhq bot pushed a commit that referenced this pull request Jan 6, 2023

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Depends on #44666

This will install `package.json` dependencies in examples before running them.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
created-by: Next.js team PRs by the Next.js team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants