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

Bump Node.js to 16 #4277

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Bump Node.js to 16 #4277

wants to merge 3 commits into from

Conversation

Mrtenz
Copy link
Collaborator

@Mrtenz Mrtenz commented Feb 8, 2022

Node.js 12 is currently maintenance only, and is soon EOL. Node.js 16 is the current LTS:

@Mrtenz Mrtenz temporarily deployed to STAGING February 8, 2022 11:19 Inactive
@FrederikBolding FrederikBolding self-requested a review February 8, 2022 11:24
@mycrypto-bot
Copy link
Collaborator

Staging build: MyCryptoBuilds

@codecov
Copy link

codecov bot commented Feb 8, 2022

Codecov Report

Merging #4277 (4836589) into master (eab9fa1) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4277   +/-   ##
=======================================
  Coverage   78.77%   78.77%           
=======================================
  Files         572      572           
  Lines       12438    12438           
  Branches     3080     3080           
=======================================
  Hits         9798     9798           
  Misses       2608     2608           
  Partials       32       32           

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 eab9fa1...4836589. Read the comment docs.

@stale
Copy link

stale bot commented Apr 11, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@samisbakedham
Copy link

This code looks good to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants