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

fix: regenerated pnpm-lock.yaml #35

Merged
merged 4 commits into from
Nov 24, 2023
Merged

Conversation

samderanova
Copy link
Contributor

This is an attempt to resolve #34 , in which Vercel throws a warning saying that the pnpm-lock.yaml file is "incompatible", by deleting and regenerating it.

@samderanova samderanova linked an issue Nov 24, 2023 that may be closed by this pull request
Copy link
Contributor

github-actions bot commented Nov 24, 2023

Deploy preview for irvinehacks-site-2024-sanity ready!

Name Sanity Studio
Preview Visit Preview
Commit fd45405

Copy link
Contributor

github-actions bot commented Nov 24, 2023

Deploy preview for irvinehacks-site-2024 ready!

Name IrvineHacks 2024 Site
Preview Visit Preview
Commit fd45405

@taesungh
Copy link
Member

Think the issue is there's a regular npm package-lock.json in apps/site which explains why this is not an issue when building the Sanity app.

Copy link
Member

@taesungh taesungh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No more error message, seems to work

Package Manager changed from "npm" to "pnpm"
...
Detected pnpm-lock.yaml version 6 generated by pnpm 8

Please squash during merge.

@samderanova samderanova merged commit 39452fa into main Nov 24, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Generate new lockfile
2 participants