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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: test on node 13 #9088

Merged
merged 2 commits into from Oct 29, 2019
Merged

chore: test on node 13 #9088

merged 2 commits into from Oct 29, 2019

Conversation

SimenB
Copy link
Member

@SimenB SimenB commented Oct 24, 2019

Summary

Node 13 is out, so let's test on it. (I haven't ran this locally 馃).

Also move typecheck, website and circus from node 10 to 12, as 12 is now LTS moved to (and merged from) #9100

Test plan

Green CI

@SimenB
Copy link
Member Author

SimenB commented Oct 24, 2019

@arcanis thoughts on the pnp test failure?

image

@arcanis
Copy link
Contributor

arcanis commented Oct 24, 2019

I'm unfortunately without laptop this week, but I can look into it next Monday! It seems that the signature of Module._findPath changed, weird 馃

@sairus2k
Copy link
Contributor

Related issue yarnpkg/yarn#7642

@SimenB
Copy link
Member Author

SimenB commented Oct 29, 2019

Manually added the change in yarnpkg/yarn#7650. A distinct advantage of checking in a version of yarn 馃憤

@codecov-io
Copy link

Codecov Report

Merging #9088 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #9088   +/-   ##
=======================================
  Coverage   64.73%   64.73%           
=======================================
  Files         277      277           
  Lines       11707    11707           
  Branches     2874     2874           
=======================================
  Hits         7578     7578           
  Misses       3512     3512           
  Partials      617      617

Continue to review full report at Codecov.

Legend - Click here to learn more
螖 = absolute <relative> (impact), 酶 = not affected, ? = missing data
Powered by Codecov. Last update cc2c92b...5501f5e. Read the comment docs.

@SimenB
Copy link
Member Author

SimenB commented Oct 29, 2019

@arcanis I assume it's safe? I just added || [], and CI passed at least

@SimenB
Copy link
Member Author

SimenB commented Oct 29, 2019

Can revert if not

@SimenB SimenB merged commit 107657d into jestjs:master Oct 29, 2019
@SimenB SimenB deleted the node-13 branch October 29, 2019 08:50
@arcanis
Copy link
Contributor

arcanis commented Oct 29, 2019

@SimenB yep, that's the right change!

Manually added the change in yarnpkg/yarn#7650. A distinct advantage of checking in a version of yarn 馃憤

And just wait until your hear about yarn set version from sources --branch <pr#> (v2 only) 馃槢

@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants