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

chore: map mobile browser data to their desktop version #10814

Merged
merged 4 commits into from Jan 24, 2020

Conversation

JLHwung
Copy link
Contributor

@JLHwung JLHwung commented Dec 4, 2019

Q                       A
Fixed Issues? Fixes issue mentioned in here
Patch: Bug Fix? Yep
Tests Added + Pass? Yes
License MIT

In this PR we map the ES Module support data for mobile browser to their desktop versions. caniuse now preserve only latest version for the mobile browser other than iOS Safari. It fixes the issue that when caniuse-db is upgraded, the and_chr and and_ff will be incorrectly bumped.

@JLHwung JLHwung added the PR: Internal 🏠 A type of pull request used for our changelog categories label Dec 4, 2019
nicolo-ribaudo
nicolo-ribaudo previously approved these changes Dec 4, 2019
@JLHwung
Copy link
Contributor Author

JLHwung commented Dec 4, 2019

@nicolo-ribaudo Actually it works like browserslist/browserslist#428 by replacing the caniuse mobile desktop agents by their desktop versions.

Oh it does depend on that since we will throw on unknown browser versions. Now we have to skip android before the upstream PR is resolved.

@nicolo-ribaudo
Copy link
Member

The upstream fix has been released

@JLHwung
Copy link
Contributor Author

JLHwung commented Dec 5, 2019

@nicolo-ribaudo unskipping android will introduce android 61 to the browser sets from esmodules: true, which does not work correctly until #10790 since android >= 37 data is absent (and of course unnecessary to be present) from compat-table.

Therefore, I suggest we merge it as-is.

@@ -120,7 +120,7 @@ describe("getTargets", () => {
it("does throws on unsupported versions", () => {
expect(() => {
getTargets({
browsers: "node 15.0.0, chrome 1000",
browsers: "node 1000.0.0, chrome 1000",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Time flies: node 15 is scheduled at Oct. 2020.

@nicolo-ribaudo nicolo-ribaudo merged commit b54a946 into babel:master Jan 24, 2020
@nicolo-ribaudo nicolo-ribaudo deleted the fix-build-module-data-script branch January 24, 2020 21:40
rajasekarm pushed a commit to rajasekarm/babel that referenced this pull request Feb 17, 2020
* chore: map mobile browser data to their desktop version

* chore: skip android until upstream support is resolved

* Revert "chore: skip android until upstream support is resolved"

This reverts commit 436e9af.

* Update fixtures

Co-authored-by: Nicolò Ribaudo <nicolo.ribaudo@gmail.com>
@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 Apr 25, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 25, 2020
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 PR: Internal 🏠 A type of pull request used for our changelog categories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants