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(rspack): remote applications e2e setup incorrectly #30273 #30531

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

Conversation

Coly010
Copy link
Contributor

@Coly010 Coly010 commented Mar 28, 2025

Current Behavior

The remote applications that are generated with an e2e project that is incorrectly set up.
There is also an issue in the @nx/rspack:module-federation-static-server where the staticRemotesPort is incorrectly calculated when the remote has no nested remotes.

Expected Behavior

Fix the issue in the executor for calculating the staticRemotesPort to ensure it is a valid port.
Ensure the correct port is set in the e2e config file

Related Issue(s)

Fixes #30273

@Coly010 Coly010 requested review from a team as code owners March 28, 2025 10:22
@Coly010 Coly010 self-assigned this Mar 28, 2025
@Coly010 Coly010 requested a review from mandarini March 28, 2025 10:22
Copy link

vercel bot commented Mar 28, 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 28, 2025 10:25am

Copy link

nx-cloud bot commented Mar 28, 2025

View your CI Pipeline Execution ↗ for commit d47a490.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 3h 56m 6s 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=4a3a8... ✅ Succeeded 4s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 3s View ↗
nx documentation ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-03-28 16:39:17 UTC

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.

@nx/react module-federation e2e command fails with freshly generated remote application
1 participant