-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Use peerDeps in register and babel-node, add missing peerDependencies, and declare devDependencies #6655
Conversation
loganfsmyth
commented
Oct 31, 2017
Q | A |
---|---|
Fixed Issues? | |
Patch: Bug Fix? | |
Major: Breaking Change? | |
Minor: New Feature? | |
Tests Added + Pass? | Yes |
Documentation PR | |
Any Dependency Changes? |
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/5567/ |
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/5561/ |
@@ -9,7 +9,7 @@ There are some shell-executable utility scripts, `babel-external-helpers.js` and | |||
## Install | |||
|
|||
```sh | |||
npm install --save-dev @babel/cli | |||
npm install --save-dev @babel/core @babel/cli |
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.
I guess we should remove babel-core in the deps for babel-cli too
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.
Woops, saw that it had the peerDep and didn't realize it still had it in dependencies
.
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.
lgtm, linting failure I fixed in my pr
ea7e643
to
2d7685b
Compare
cool, feel free to merge |