Content-Length: 298841 | pFad | https://redirect.github.com/babel/babel/pull/16907

E7 fix: support BROWSERSLIST{,_CONFIG} env by JLHwung · Pull Request #16907 · 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.

Sign up for GitHub

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

Jump to bottom

fix: support BROWSERSLIST{,_CONFIG} env #16907

Merged
merged 1 commit into from
Oct 17, 2024
Merged

fix: support BROWSERSLIST{,_CONFIG} env #16907

merged 1 commit into from
Oct 17, 2024

Conversation

JLHwung
Copy link
Contributor

@JLHwung JLHwung commented Oct 15, 2024

Q                       A
Fixed Issues? Fixes #16906
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

Fixes a regression introduced in #16858. Thanks @isemanjski for reporting this bug.

@JLHwung JLHwung added PR: Bug Fix 🐛 A type of pull request used for our changelog categories i: regression labels Oct 15, 2024
@babel-bot
Copy link
Collaborator

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/58120

@nicolo-ribaudo
Copy link
Member

Should we read those env vars through api.env(), so that they will be accounted for when considering preset caching?

@JLHwung
Copy link
Contributor Author

JLHwung commented Oct 16, 2024

Should we read those env vars through api.env(), so that they will be accounted for when considering preset caching?

Good question! I don't know api.env() can read process.env until today, for this case we will have to pass through the Preset/Plugin API to the compilation helper, or pass through an env reader. I am a bit hesitant passing through the API interface as the helper will be strongly coupled with plugin/preset, making it more difficult to be used by the babel core.

This PR is to restore the previous behaviour before #16858 when we call the browserslist.loadConfig(). Reading through api.env() can definitely an idea for a new PR.

@nicolo-ribaudo nicolo-ribaudo merged commit d658738 into babel:main Oct 17, 2024
52 checks passed
@nicolo-ribaudo nicolo-ribaudo deleted the fix-16906 branch October 17, 2024 09:21
@davidjbradshaw davidjbradshaw mentioned this pull request Oct 26, 2024
Merged
@DevangPatelUK DevangPatelUK mentioned this pull request Nov 12, 2024
Open
This was referenced Nov 12, 2024
Open
Open
@ali8889 ali8889 mentioned this pull request Nov 13, 2024
Open
@sebastianthulin sebastianthulin mentioned this pull request Nov 13, 2024
Merged
@DanielRivers DanielRivers mentioned this pull request Nov 13, 2024
Merged
@maismas maismas mentioned this pull request Nov 13, 2024
Merged
@NOUIY NOUIY mentioned this pull request Nov 13, 2024
Open
@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.
Reviewers

@liuxingbaoyu liuxingbaoyu liuxingbaoyu approved these changes

@nicolo-ribaudo nicolo-ribaudo nicolo-ribaudo approved these changes

Assignees
No one assigned
Labels
i: regression outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Bug Fix 🐛 A type of pull request used for our changelog categories
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[regression] Usage of the BROWSERSLIST env var
4 participants
@JLHwung @babel-bot @nicolo-ribaudo @liuxingbaoyu








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://redirect.github.com/babel/babel/pull/16907

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy