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(core): fix local registry not being considered when using bun p… #30459

Merged
merged 1 commit into from
Mar 23, 2025

Conversation

tinesoft
Copy link
Contributor

@tinesoft tinesoft commented Mar 22, 2025

…ackage manager

Current Behavior

When using bun package manager, the local registry url set when starting verdaccio is ignored by bun, which continues to point to default NPM registry, which in return causes an error when trying to publish package

> nx run common:nx-release-publish

bun publish error:
error: missing authentication (run `bunx npm login`)

Expected Behavior

Verdaccio should work when using bun pm

Related Issue(s)

Fixes #

@tinesoft tinesoft requested review from meeroslav, vsavkin, mandarini and a team as code owners March 22, 2025 07:46
@tinesoft tinesoft requested a review from FrozenPandaz March 22, 2025 07:46
Copy link

vercel bot commented Mar 22, 2025

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

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview Mar 23, 2025 2:02pm

Copy link

nx-cloud bot commented Mar 22, 2025

View your CI Pipeline Execution ↗ for commit 3798d22.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 42m 32s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 17s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check --base=3be73... ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 1s View ↗
nx documentation ✅ Succeeded 1m 33s View ↗

☁️ Nx Cloud last updated this comment at 2025-03-23 14:48:49 UTC

@tinesoft tinesoft force-pushed the fix/verdaccio-with-bun branch from 758b8f9 to 3798d22 Compare March 23, 2025 14:00
tinesoft added a commit to tinesoft/nx that referenced this pull request Mar 23, 2025
…om registries (i.e `verdaccio`)

This is not longer necessary, thanks to nrwl#30459
tinesoft added a commit to tinesoft/nx that referenced this pull request Mar 23, 2025
…om registries (i.e `verdaccio`)

This is not longer necessary, thanks to nrwl#30459
Copy link
Collaborator

@JamesHenry JamesHenry left a comment

Choose a reason for hiding this comment

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

We obviously haven't been able to test this working on our side, but I trust you 😄

@JamesHenry JamesHenry merged commit cecd607 into nrwl:master Mar 23, 2025
12 checks passed
tinesoft added a commit to tinesoft/nx that referenced this pull request Mar 23, 2025
…om registries (i.e `verdaccio`)

This is not longer necessary, thanks to nrwl#30459
FrozenPandaz pushed a commit that referenced this pull request Mar 25, 2025
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