-
Notifications
You must be signed in to change notification settings - Fork 12k
ng deploy should not recommend @azure/ng-deploy as an example #23255
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
Comments
Checked with @mgechev and we should remove NB: This change also needs to be done in AIO https://github.com/angular/angular/blob/68a6a075f4aeab1788a1bba736bfcb428cfbbacb/aio/content/guide/deployment.md |
Thank for the heads-up @alan-agius4 @mgechev. In all transparency, the core maintainer of that package has left our team. I am working with the rest of the team to plan a better maintenance strategy for this package. I am prioritizing this work for the next week and will keep you posted. I apologize to everyone for the inconvenience, on behalf of the whole team. |
This builder is not compatible with newer versions of the Angular CLI. See angular#23255 for more context Closes angular#23255
This builder is not compatible with newer versions of the Angular CLI. See angular/angular-cli#23255 for more context
This builder is not compatible with newer versions of the Angular CLI. See angular/angular-cli#23255 for more context PR Close #46285
This builder is not compatible with newer versions of the Angular CLI. See angular/angular-cli#23255 for more context PR Close #46285
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
🐞 Bug report
Command (mark with an
x
)Is this a regression?
No
Description
The
@azure/ng-deploy
is outdated and only support Angular 9 at this moment. I think this one should be removed from the example to avoid confusion to newbies.🔬 Minimal Reproduction
🔥 Exception or Error
🌍 Your Environment
Anything else relevant?
This issue is related to #22978 and microsoft/playwright#11258
The text was updated successfully, but these errors were encountered: