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(misc): conditionally allow starter template inclusion #14268

Merged

Conversation

DirkoOdendaal
Copy link
Contributor

Current Behavior

When generating a new angular applications it always adds a NX welcome component to the application.
For us this is something we don't want, so we always delete the nx welcome component once it's created.

Expected Behavior

That the application generator has a flag to determine if a nx welcome component should be created or not.
This will speed up the generator itself if no component needs to be created or added.

@vercel
Copy link

vercel bot commented Jan 11, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
nx-dev ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 27, 2023 at 1:42PM (UTC)

@DirkoOdendaal
Copy link
Contributor Author

Hi @Coly010 , @FrozenPandaz ,

I recreated this PR to master, hope all is good with it now.

@jaysoo
Copy link
Member

jaysoo commented Jan 20, 2023

Hi @DirkoOdendaal , thanks for your PR.

Can you change it to --minimal flag rather than --skipStarterTemplate? We use "minimal" as a catch-all for generating a clean setup that doesn't include things like default welcome screen.

@DirkoOdendaal
Copy link
Contributor Author

Hi @jaysoo ,
No problem, updated.

@github-actions
Copy link

github-actions bot commented Mar 5, 2023

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 Mar 5, 2023
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.

3 participants