Content-Length: 212688 | pFad | https://github.com/babel/babel/issues/16906

A6 [regression] Usage of the BROWSERSLIST env var · Issue #16906 · babel/babel · GitHub
Skip to content
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

[regression] Usage of the BROWSERSLIST env var #16906

Closed
nicolo-ribaudo opened this issue Oct 15, 2024 · 0 comments · Fixed by #16907
Closed

[regression] Usage of the BROWSERSLIST env var #16906

nicolo-ribaudo opened this issue Oct 15, 2024 · 0 comments · Fixed by #16907
Labels
outdated A closed issue/PR that is archived due to age. Recommended to make a new issue

Comments

@nicolo-ribaudo
Copy link
Member

nicolo-ribaudo commented Oct 15, 2024

Hi, it seems that this change broke the resolution of browsers from BROWSERSLIST environment variable. Because you are now conditionally calling browserslist.loadConfig only if configFile exists. But there is a case when someone might not use any config file option but only the env variable. In loadConfig method the first that browserslist library does is to return the value of process.env.BROWSERSLIST if it's defined, otherwise, it proceeds with resolving other types of configurations.

Originally posted by @isemanjski in #16858 (comment)

@github-actions github-actions bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Jan 17, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated A closed issue/PR that is archived due to age. Recommended to make a new issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/babel/babel/issues/16906

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy