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

feat(workspace-plugin): add generation regexp check in order to prevent running generators with invalid project name including npm scope #33968

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

Conversation

Hotell
Copy link
Contributor

@Hotell Hotell commented Mar 11, 2025

  • NOTE: all monorepo project names exclude npmScope in order to mitigate churn and verbose typing
  • if you decide to avoid using prompts that we ship with our generators, or yarn nx generate and provide --project name explicitly you may by accident provide the project name with npmScope

Previous Behavior

yarn nx generate @fluentui/workspace-plugin:prepare-initial-release --project=@fluentui/react-color-picker-preview --phase=stable

image

New Behavior

yarn nx generate @fluentui/workspace-plugin:prepare-initial-release --project=@fluentui/react-color-picker-preview --phase=stable

image

Related Issue(s)

  • Fixes #

…nt running generators with invalid project name including npm scope
@github-actions github-actions bot added this to the March Project Cycle Q1 2025 milestone Mar 11, 2025
@Hotell Hotell marked this pull request as ready for review March 11, 2025 14:06
@Hotell Hotell requested a review from a team as a code owner March 11, 2025 14:06
Copy link

📊 Bundle size report

✅ No changes found

Copy link

Pull request demo site: URL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant