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
[preset-env
] Add browserslistEnv
option
#11434
Conversation
The failing tests should be fixed on master. |
value = defaultValue; | ||
} | ||
|
||
if (typeof value !== "string" && typeof value !== "undefined") { |
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.
if (typeof value !== "string" && typeof value !== "undefined") { | |
else if (typeof value !== "string") { |
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
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 with some nits.
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.
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.
@AndrewLeedham nice work!
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 a71b140:
|
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/22657/ |
preset-env
] Add browserslistEnv
option
Adds a new option to
preset-env
, which allows the user to specify an environment which is passed forward to browserslist. Continuation of the work from #9161 based on this comment: #9161 (comment)