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(angular): add generator to convert targets to use the esbuild-based application executor #21333

Conversation

leosvelperez
Copy link
Member

@leosvelperez leosvelperez commented Jan 25, 2024

Adds the convert-to-application-executor generator that allows converting a project or all projects using one of the @angular-devkit/build-angular:browser, @angular-devkit/build-angular:browser-esbuild, @nx/angular:browser and @nx/angular:browser-esbuild executors to use the @nx/angular:application executor or the @angular-devkit/build-angular:application builder. If the converted target is using one of the @nx/angular executors, the @nx/angular:application executor will be used. Otherwise, the @angular-devkit/build-angular:application builder will be used.

Some restrictions apply and the project(s) won't be converted if any of the following conditions is met:

  • Target uses a custom webpack configuration
  • Target uses the unsupported deployUrl option
  • There is more than one target for the browser or server builds

Additionally, the generator can only be run on workspaces using Angular v17.0.0 or greater.

Current Behavior

Expected Behavior

Related Issue(s)

Fixes #

@leosvelperez leosvelperez self-assigned this Jan 25, 2024
Copy link

vercel bot commented Jan 25, 2024

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

1 Ignored Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview Jan 30, 2024 2:58pm

@leosvelperez leosvelperez force-pushed the angular/convert-to-application-executor-generator branch 4 times, most recently from 020f4b5 to 4a7f91e Compare January 30, 2024 11:25
@leosvelperez leosvelperez marked this pull request as ready for review January 30, 2024 11:25
@leosvelperez leosvelperez force-pushed the angular/convert-to-application-executor-generator branch from 4a7f91e to c8c2432 Compare January 30, 2024 14:57
@leosvelperez leosvelperez merged commit 7a77b0d into nrwl:master Jan 30, 2024
6 checks passed
@leosvelperez leosvelperez deleted the angular/convert-to-application-executor-generator branch January 30, 2024 16:27
FrozenPandaz pushed a commit that referenced this pull request Jan 31, 2024
…sed application executor (#21333)

(cherry picked from commit 7a77b0d)
Copy link

github-actions bot commented Feb 5, 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 Feb 5, 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.

None yet

3 participants