Skip to content

Issue with release 6.0.4 #11022

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

Closed
Brocco opened this issue May 25, 2018 · 2 comments
Closed

Issue with release 6.0.4 #11022

Brocco opened this issue May 25, 2018 · 2 comments

Comments

@Brocco
Copy link
Contributor

Brocco commented May 25, 2018

Post mortem of the issue with release 6.0.4

What happened

Version 6.0.4 of the Angular CLI was published to npm on May 24. After it was released reports of issues with the new and generate commands were reported. As a result the decision was made to release version 6.0.5 which is the same version as 6.0.3, essentially rolling back release 6.0.4.

What went wrong

Tracing down the issue led to this PR angular/devkit#899 which restricted the properties that can be sent to the default schematics to only those defined within the schema. The CLI removes non-schematic specific options so they are not passed through to the schematic engine, which allows schematics to specify the additionalProperties value to be false. A bug in the CLI was not removing the dryRun option because it was being defined as dry-run and the property dryRun was the one being removed. This caused the dryRun property to be passed to schematics which now have additionalProperties set to false, causing the reported issues.

How is it being addressed

The bug in the CLI has been addressed in PR #11015 which renamed the property definition to dryRun which will allow proper filtering of the non-schematic options. The other step to address this is as this logic is changing the public API of the schematics, it will be included in a future minor release.

What is being done to prevent it

Integration of the CI from the angular/devkit repo to also run the angular/angular-cli CI would catch errors like this in the future.

@filipesilva
Copy link
Contributor

Closing as this has been sorted now, and the issue has been up for a while for visibility.

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants