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

Add registry option to publishConfig #6775

Closed
WhiteKiwi opened this issue Jul 6, 2023 · 2 comments · Fixed by #6803
Closed

Add registry option to publishConfig #6775

WhiteKiwi opened this issue Jul 6, 2023 · 2 comments · Fixed by #6803

Comments

@WhiteKiwi
Copy link
Sponsor Contributor

WhiteKiwi commented Jul 6, 2023

Describe the user story

I proceeded with the registry migration using nexus group.

I did group legacy-registry and new-registry. 
group-registry is used for reading.
new-registry is used for uploading.

There are currently two ways to solve this issue.
1. modify pnpm registry config just before uploading
2. set publicConfig.registry and using npm

I want to use publishConfig.registry in pnpm because method 2 looks neat.
Currently, I use npm only for publishing.

Describe the solution you'd like

This issue is resolved if pnpm supports publishConfig.registry.

Describe the drawbacks of your solution

Additional work is needed, and I don't think there will be many situations where read repo and write repo are different.

Describe alternatives you've considered

The alternative is described above.

It is written using a translator. Please comment if you don't understand anything.
Thank you for reading.

@KSXGitHub
Copy link
Contributor

pnpm just calls npm publish though. Do publishConfig not get passed to npm? Have you tried it?

@WhiteKiwi
Copy link
Sponsor Contributor Author

I was late to check
thank you!

zkochan pushed a commit that referenced this issue Jul 17, 2023
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.

2 participants