Skip to content

Parsing fixes#12376

Merged
hansl merged 3 commits intoangular:masterfrom
hansl:fix-12308
Sep 27, 2018
Merged

Parsing fixes#12376
hansl merged 3 commits intoangular:masterfrom
hansl:fix-12308

Conversation

@hansl
Copy link
Copy Markdown
Contributor

@hansl hansl commented Sep 26, 2018

fix(@angular/cli): allow -a=value arguments …

If a flag is followed by an equal sign, just treat it as a long name. So
the example above would translate to --a=value, while -abc=123 would be
"-a -b -c=123".

fix(@angular/cli): numerical flags should not give 0 if empty …

If the value is an empty string, a number conversion would give 0. It is unexpected
from the user standpoint ("--num=" has the user expect a string value).

Fixes #12308

If a flag is followed by an equal sign, just treat it as a long name. So
the example above would translate to --a=value, while -abc=123 would be
"-a -b -c=123".

Fixes angular#12308
And numerical positional flags will be ignored.

If the value is an empty string, a number conversion would give 0. It is unexpected
from the user standpoint ("--num=" has the user expect a string value).
@hansl hansl added the target: major This PR is targeted for the next major release label Sep 26, 2018
@hansl hansl requested a review from clydin September 26, 2018 20:51
@hansl hansl merged commit 5faf0cb into angular:master Sep 27, 2018
@hansl hansl deleted the fix-12308 branch November 29, 2018 01:16
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

No changes were made during component generation

3 participants