-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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(module-federation): ensure @nx/module-federation package is installed for users #29416
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
View your CI Pipeline Execution ↗ for commit 4ef4c0d.
☁️ Nx Cloud last updated this comment at |
packages/angular/src/migrations/update-20-2-0/migrate-with-mf-import-to-new-package.ts
Show resolved
Hide resolved
7f385c1
to
4ef4c0d
Compare
@@ -5,6 +5,12 @@ | |||
"version": "20.2.0-beta.3", | |||
"description": "Update the withModuleFederation import use @nx/module-federation/rspack.", | |||
"factory": "./src/migrations/update-20-2-0/migrate-with-mf-import-to-new-package" | |||
}, | |||
"ensure-nx-module-federation-package": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is technically already handled by Angular and React plugins right? Is there a scenario where those aren't installed by Rspack is? We don't do this for Webpack.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Each plugin is finding its own set of module-federation-dev-server
executors to determine if MF projects exists.
I could combine the rspack ones into the react one, i just had them separated based on location of the executor
…lled for users (#29416) ## Current Behavior The `@nx/module-federation` package is a direct dependency of some of the other packages in the Nx Plugin ecosystem. It should be resolved correctly by package managers such that it can be used when setting up MF projects. However, some users are facing issues with module resolutions where the package is not found as expected. ## Expected Behavior Install the package directly for the user to ensure module resolution works as expected. ## Related Issue(s) Fixes #29269
Current Behavior
The
@nx/module-federation
package is a direct dependency of some of the other packages in the Nx Plugin ecosystem.It should be resolved correctly by package managers such that it can be used when setting up MF projects.
However, some users are facing issues with module resolutions where the package is not found as expected.
Expected Behavior
Install the package directly for the user to ensure module resolution works as expected.
Related Issue(s)
Fixes #29269