Skip to content

Add pnpm.onlyBuiltDependencies field to package.json#1016

Open
holazz wants to merge 2 commits into
vercel:mainfrom
holazz:fix/pnpm
Open

Add pnpm.onlyBuiltDependencies field to package.json#1016
holazz wants to merge 2 commits into
vercel:mainfrom
holazz:fix/pnpm

Conversation

@holazz

@holazz holazz commented Mar 1, 2025

Copy link
Copy Markdown

pnpm 10 blocks lifecycle scripts by default.

https://github.com/pnpm/pnpm/releases/tag/v10.0.0

@vercel

vercel Bot commented Mar 1, 2025

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
next-learn-starter ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 3, 2025 2:31pm
next-seo-starter ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 3, 2025 2:31pm

@vercel

vercel Bot commented Mar 1, 2025

Copy link
Copy Markdown
Contributor

@holazz is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@w0244079

w0244079 commented Mar 3, 2025

Copy link
Copy Markdown

@holazz I can confirm also that this is needed....however when i run "pnpm approve-builds" to add the onlyBuiltDependencies it also lists the 'sharp' library...I'm assuming for image manipulation...

"pnpm": {
"onlyBuiltDependencies": [
"bcrypt",
"sharp"
]
}

might be good to add sharp as well.

Thanks!

@holazz

holazz commented Mar 3, 2025

Copy link
Copy Markdown
Author

@holazz I can confirm also that this is needed....however when i run "pnpm approve-builds" to add the onlyBuiltDependencies it also lists the 'sharp' library...I'm assuming for image manipulation...

"pnpm": { "onlyBuiltDependencies": [ "bcrypt", "sharp" ] }

might be good to add sharp as well.

Thanks!

Yes!

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.

2 participants