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

cleanup(testing): remove bun's workaround to make it work with cust… #30468

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tinesoft
Copy link
Contributor

@tinesoft tinesoft commented Mar 23, 2025

…om registries (i.e verdaccio)

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 #

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

Copy link

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 meeroslav, vsavkin, mandarini and a team 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
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