-
Notifications
You must be signed in to change notification settings - Fork 2
Highcharts version bump #8
base: master
Are you sure you want to change the base?
Conversation
package.json
Outdated
| "version": "0.6.1", | ||
| "description": "Highcharts for your Angular2 project", | ||
| "scripts": { | ||
| "clean": "./node_modules/.bin/rimraf", |
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.
you only need to write "rimraf" in here (it automatically looks inside the .bin folder
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 about that
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.
done
package.json
Outdated
| "clean": "./node_modules/.bin/rimraf", | ||
| "watch": "ntsc -w", | ||
| "build": "rm -rf dist && ngc", | ||
| "build": "npm run clean -rf dist && ngc", |
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.
Isn't the "-rf" implied by the rimraf module?
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.
yeah I missed that one. It is fixed in the other commit^^
No description provided.