-
Notifications
You must be signed in to change notification settings - Fork 256
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 (schematics): allow comments in angular.json file (Issue #480) #550
Conversation
It seems this pr is stuck. May be because replacing pr #531 just after @SanderElias assigned the review of this last one to @jorgeucano. |
libs/scully-schematics/package.json
Outdated
@@ -15,7 +15,9 @@ | |||
"test": "tsc -p tsconfig.spec.json && jasmine src/**/*_spec.js", | |||
"publish:patch": "tsc -p ./tsconfig.json && npm version patch && npm publish --access public", | |||
"publish:minor": "tsc -p ./tsconfig.json && npm version minor && npm publish --access public", | |||
"publish:major": "tsc -p ./tsconfig.json && npm version major && npm publish --access public" | |||
"publish:major": "tsc -p ./tsconfig.json && npm version major && npm publish --access public", | |||
"clean": "rimraf spec/**/*.js spec/**/*.d.ts spec/**/*.js.map src/**/*.js src/**/*.d.ts src/**/*.js.map", |
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.
Why you add this?
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.
It's just a dev convenience to get a clean state before each build on workstation. Not needed during CI Build.
Can be removed if you wish.
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.
please remove this and I will approve! thanks!
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.
@jorgeucano Done
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: #480
What is the new behavior?
Allow comments in angular.json file with Scully schematics
In fact allow json5 syntax as Angular itself.
Note. ATM json5 is allowed only with angular.json files. But it'd be very easy to extend it to any json file of your choice.
Does this PR introduce a breaking change?
Other information
This is a sequel of pr #531, as in the meantime the folder schematics/scully has been moved to libs/scully-schematics.