Skip to content

Conversation

@tinesoft
Copy link
Contributor

@tinesoft tinesoft commented Mar 23, 2025

This workaround is no longer necessary once is merged #30459.

Until then, some e2e tests should still fail due to bun publish errors (which is normal).
The PR will be converted to final, once the previous is merged to master

Current Behavior

See #30459

Expected Behavior

See #30459

Related Issue(s)

Fixes #

@vercel
Copy link

vercel bot commented Mar 23, 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 25, 2025 8:34pm

@nx-cloud
Copy link
Contributor

nx-cloud bot commented Mar 23, 2025

View your CI Pipeline Execution ↗ for commit 5a22f17.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 43m 14s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 18s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check --base=50561... ✅ Succeeded 1s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗
nx documentation ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-03-25 21:19:48 UTC

@tinesoft tinesoft force-pushed the chore/remove-bun-publih-workaround branch from bf6760c to 6abab61 Compare March 23, 2025 21:55
@tinesoft tinesoft marked this pull request as ready for review March 23, 2025 21:56
@tinesoft tinesoft requested review from a team, mandarini, meeroslav and vsavkin as code owners March 23, 2025 21:56
@tinesoft tinesoft requested a review from AgentEnder March 23, 2025 21:56
@JamesHenry JamesHenry marked this pull request as draft March 23, 2025 22:13
@JamesHenry
Copy link
Collaborator

@tinesoft as noted on discords this is not ready

@tinesoft
Copy link
Contributor Author

tinesoft commented Mar 23, 2025

Ah yes, sorry! I was too quick into converting the draft pr into a final one ^^

Will convert it back into a draft again, until then

@tinesoft tinesoft force-pushed the chore/remove-bun-publih-workaround branch 2 times, most recently from 806a5b3 to 6943017 Compare March 25, 2025 12:47
…om registries (i.e `verdaccio`)

This is not longer necessary, thanks to nrwl#30459
@tinesoft tinesoft force-pushed the chore/remove-bun-publih-workaround branch from 6943017 to 5a22f17 Compare March 25, 2025 20:32
@tinesoft tinesoft marked this pull request as ready for review March 25, 2025 21:27
@JamesHenry
Copy link
Collaborator

Great, thanks again for this!

@JamesHenry JamesHenry merged commit d9cb931 into nrwl:master May 1, 2025
12 checks passed
@JamesHenry JamesHenry changed the title cleanup(testing): remove bun's workaround to make it work with cust… chore(repo): remove bun's workaround to make it work with verdaccio May 1, 2025
@github-actions
Copy link
Contributor

github-actions bot commented May 7, 2025

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants