-
-
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
Improve error messages around pipeline option #8279
Conversation
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/8526/ |
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/8513/ |
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.
Thoughts about linking to the docs in the error message?
👍
" You must pass the 'pipelineProposal' option to" + | ||
" @babel/preset-stage-0", | ||
" You must pass 'pipelineProposal' option to" + | ||
" @babel/preset-stage-0 with one of: " + |
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.
with
-> whose value must be
2f3dd14
to
4cd87d6
Compare
Updated! |
"The pipeline operator plugin requires a 'proposal' option." + | ||
"'proposal' must be one of: " + | ||
proposals.join(", ") + | ||
". More details: https://babeljs.io/docs/en/next/babel-plugin-syntax-pipeline-operator.html", |
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 think you want to link to https://babeljs.io/docs/en/next/babel-plugin-proposal-pipeline-operator? If not, then we need the doc in the syntax plugin, and you can remove the .html
now
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.
Whoops, fixed.
4cd87d6
to
db9819d
Compare
@mAAdhaTTah after this branch merged to master, my error message still like this:
|
Can you try to redownload and rebuild the repo? |
Improve the error messages so we (hopefully) get fewer issues about this.
Thoughts about linking to the docs in the error message? Good or bad idea?