-
Notifications
You must be signed in to change notification settings - Fork 12k
feat(@angular/cli): add flags to the process title #11920
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
Conversation
packages/angular/cli/bin/ng
Outdated
|
||
var version = process.version.substr(1).split('.'); | ||
const version = process.version.substr(1).split('.'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can potentially be run from an old version of node which doesn’t support const. It would crash instead of displaying the error message
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah. I was wondering why that was there. Fixed.
4ccaa04
to
958e54e
Compare
958e54e
to
2625ca8
Compare
@hansl ah great! Looking forward to seeing it live in CLI! Thanks and have a great day! |
c7a2a2f
to
04ca274
Compare
Closes angular#8772 Fixes angular#6888 Special thanks to @tomastrajan for original PR.
04ca274
to
93bdf75
Compare
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. |
Closes #8772
Fixes #6888
Special thanks to @tomastrajan for original PR.