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

Update browserslist/caniuse-lite #13605

Merged
merged 9 commits into from May 31, 2020

Conversation

@Janpot
Copy link
Collaborator

@Janpot Janpot commented May 31, 2020

Looks like caniuse-lite is out of date and causing test failures.

  • I upgraded both browserslist and caniuse-lite to latest semver compatible version.
  • This seemed to cause changes in ncc compiled files, so recompiled.
  • lint-staged failed on these files even though they should be ignored. As a fix, I applied the advice from https://github.com/okonet/lint-staged#how-can-i-ignore-files-from-eslintignore-
  • Updated some test snapshots. 🤔 not sure this is the way to go
@Janpot Janpot requested a review from Timer as a code owner May 31, 2020
@Janpot Janpot requested review from ijjk, lfades and timneutkens as code owners May 31, 2020
Janpot added 5 commits May 31, 2020
@Timer Timer added this to the 9.4.5 milestone May 31, 2020
@Timer Timer added the type: next label May 31, 2020
@Timer
Timer approved these changes May 31, 2020
@kodiakhq kodiakhq bot merged commit f697574 into vercel:canary May 31, 2020
19 of 21 checks passed
19 of 21 checks passed
build
Details
PR Stats
Details
lint
Details
Check Pre-compiled
Details
Test All (1)
Details
Test All (2)
Details
Test All (3)
Details
Test All (4)
Details
Test All (5)
Details
Test All (6)
Details
Test Firefox (production)
Details
Test Safari (production)
Details
thank you, next
Details
Test Safari 10.1 (nav)
Details
azure next.js Build #20200531.22 had test failures
Details
azure next.js (test_chrome node-10-2) test_chrome node-10-2 was canceled
Details
kodiakhq: status ⛴ attempting to merge PR (merging)
Details
azure next.js (test_chrome node-10-1) test_chrome node-10-1 succeeded
Details
azure next.js (test_chrome node-10-3) test_chrome node-10-3 succeeded
Details
azure next.js (test_chrome node-10-4) test_chrome node-10-4 succeeded
Details
azure next.js (test_ie11) test_ie11 succeeded
Details
rokinsky added a commit to rokinsky/next.js that referenced this pull request Jul 11, 2020
Looks like `caniuse-lite` is out of date and causing test failures. 
- I upgraded both `browserslist` and `caniuse-lite` to latest semver compatible version.
- This seemed to cause changes in ncc compiled files, so recompiled.
- `lint-staged` failed on these files even though they should be ignored. As a fix, I applied the advice from https://github.com/okonet/lint-staged#how-can-i-ignore-files-from-eslintignore-
- Updated some test snapshots. 🤔 not sure this is the way to go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants