Skip to content

feat(module-federation): consolidate module federation utils into module-federation package#28919

Merged
Coly010 merged 9 commits intomasterfrom
module-federation/move-utils
Nov 18, 2024
Merged

feat(module-federation): consolidate module federation utils into module-federation package#28919
Coly010 merged 9 commits intomasterfrom
module-federation/move-utils

Conversation

@Coly010
Copy link
Copy Markdown
Contributor

@Coly010 Coly010 commented Nov 13, 2024

  • feat(module-federation): consolidate module federation utils into module-federation package
  • chore(module-federation): fix tests and linting

Current Behavior

Our current support for Module Federation relies on utilities that are spread and duplicated across the @nx/webpack package and the @nx/rspack package.

Expected Behavior

Now that we have a @nx/module-federation package, dedupe the utils and consolidate them into a single package

Todo

  • Migrations for React + Angular to install @nx/module-federation and point ModuleFederationConfig export to that package from webpack.config and rspack.config files

@Coly010 Coly010 requested review from a team, FrozenPandaz and jaysoo as code owners November 13, 2024 14:00
@Coly010 Coly010 self-assigned this Nov 13, 2024
@Coly010 Coly010 requested a review from vsavkin as a code owner November 13, 2024 14:00
@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 13, 2024

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 Nov 15, 2024 4:03pm

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Nov 13, 2024

@Coly010 Coly010 marked this pull request as draft November 13, 2024 15:36
@Coly010 Coly010 force-pushed the module-federation/move-utils branch from 495da72 to 09239f2 Compare November 14, 2024 10:04
@Coly010 Coly010 force-pushed the module-federation/move-utils branch from 462a72c to fadd817 Compare November 14, 2024 15:21
@Coly010 Coly010 marked this pull request as ready for review November 14, 2024 15:46
@Coly010 Coly010 requested a review from AgentEnder as a code owner November 14, 2024 15:46
Comment thread packages/angular/src/utils/mf/with-module-federation.ts
Comment thread packages/module-federation/src/utils/models/index.ts Outdated
Comment thread packages/module-federation/src/utils/share.ts Outdated
Comment thread packages/react/src/migrations/update-20-2-0/migrate-mf-imports-to-new-package.ts Outdated
@Coly010 Coly010 requested a review from leosvelperez November 15, 2024 10:57
@Coly010 Coly010 force-pushed the module-federation/move-utils branch from 524d0b1 to 39039ec Compare November 15, 2024 11:23
Comment thread packages/react/src/module-federation/with-module-federation.ts Outdated
@Coly010 Coly010 force-pushed the module-federation/move-utils branch from 39039ec to 60a83cc Compare November 15, 2024 16:00
@Coly010 Coly010 enabled auto-merge (squash) November 15, 2024 16:08
@Coly010 Coly010 merged commit 76d61ea into master Nov 18, 2024
@Coly010 Coly010 deleted the module-federation/move-utils branch November 18, 2024 19:15
@github-actions
Copy link
Copy Markdown
Contributor

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 Nov 24, 2024
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.

5 participants