Skip to content

fix(module-federation): normalize kebab-cased names to snake_cased#28237

Merged
Coly010 merged 1 commit intomasterfrom
module-federation/ensure-valid-remote-name
Oct 2, 2024
Merged

fix(module-federation): normalize kebab-cased names to snake_cased#28237
Coly010 merged 1 commit intomasterfrom
module-federation/ensure-valid-remote-name

Conversation

@Coly010
Copy link
Copy Markdown
Contributor

@Coly010 Coly010 commented Oct 2, 2024

Current Behavior

Nx project names can be kebab-cased but module federation remotes that use global or var will not support this.
They need to be snake cased

Expected Behavior

snake_case the remote names

Related Issue(s)

Fixes #

@Coly010 Coly010 self-assigned this Oct 2, 2024
@Coly010 Coly010 requested a review from JamesHenry as a code owner October 2, 2024 12:20
@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 2, 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 Oct 2, 2024 0:23am

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Oct 2, 2024

@Coly010 Coly010 merged commit 95d26c6 into master Oct 2, 2024
@Coly010 Coly010 deleted the module-federation/ensure-valid-remote-name branch October 2, 2024 13:22
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 8, 2024

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 Oct 8, 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.

2 participants