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

Support pnpm publish --provenance #6435

Closed
1 task done
himself65 opened this issue Apr 19, 2023 · 6 comments · Fixed by #6436
Closed
1 task done

Support pnpm publish --provenance #6435

himself65 opened this issue Apr 19, 2023 · 6 comments · Fixed by #6436

Comments

@himself65
Copy link
Contributor

Describe the user story

https://docs.npmjs.com/generating-provenance-statements
https://github.blog/2023-04-19-introducing-npm-package-provenance/

@styfle
Copy link

styfle commented Apr 19, 2023

The linked blog also mentions npm audit signatures so that will likely need to be implemented too

@dominikg
Copy link

according to https://docs.npmjs.com/generating-provenance-statements#using-third-party-package-publishing-tools adding provenance=true to .npmrc could already enable it with current pnpm even without support for the cli flag.

@styfle
Copy link

styfle commented Apr 21, 2023

Also setting the env var export NPM_CONFIG_PROVENANCE=true should work too

@filiptammergard
Copy link

according to https://docs.npmjs.com/generating-provenance-statements#using-third-party-package-publishing-tools adding provenance=true to .npmrc could already enable it with current pnpm even without support for the cli flag.

Anyone tried?

The same page says "Note: At this time, lerna and yarn are not supported tools for publishing your packages with provenance."

I wonder if pnpm is excluded from the "list" because it is supported or for other reasons.

@danielroe
Copy link

danielroe commented Apr 21, 2023

To confirm, pnpm works with setting NPM_CONFIG_PROVENANCE=true. Working commit: nuxt/nuxt@ec15037.

@himself65
Copy link
Contributor Author

To confirm, pnpm works with setting NPM_CONFIG_PROVENANCE=true. Working commit: nuxt/nuxt@ec15037.

This works for me, see toeverything/blocksuite@b6cb911

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants