Class features loose should have precedence over preset-env#11634
Conversation
|
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/23191/ |
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit cc1e1c9:
|
| if (loose) file.set(looseKey, file.get(looseKey) | feature); | ||
| if ( | ||
| loose === | ||
| "#__internal__@babel/preset-env__prefer-true-but-false-is-ok-if-it-prevents-an-error" |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
…s.js [skip ci] Co-authored-by: Huáng Jùnliàng <jlhwung@gmail.com>
JLHwung
left a comment
There was a problem hiding this comment.
Learned a good lesson when we have option constraints between plugins.
|
Thought about this a bit... I think this is fine. Can we possibly emit a warning when the loose mode from |
|
@existentialism I have added the warning. You can see in the tests added in the last commit that it can still be explicitly silenced by explicitly enabling the plugins with the correct loose mode. |
|
Is it possible to show them which plugin overrode? I think something like:
Might be slightly more helpful in both letting them know what is going on, and how to fix it. |
|
Better? |
|
I think it is, you? We might be able to shorten it, but thats more bikeshedding for later. I think it describes the problem well and how to fix it. |
|
Yeah, we don't need to shorten it. It's more important to exactly explain what's going on and how to fix it. |
Please suggest me a better method to fix this if possible 😭