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

Upgrade node to 16 #80

Merged
merged 4 commits into from
Nov 17, 2022
Merged

Upgrade node to 16 #80

merged 4 commits into from
Nov 17, 2022

Conversation

ramprasadagarwal
Copy link
Contributor

Context

Node 12 is dead since April 2022, GitHub action is currently giving a warning and asking for an upgrade.

Checklist

@s0 s0 enabled auto-merge October 23, 2022 18:28
auto-merge was automatically disabled October 23, 2022 18:39

Head branch was pushed to by a user without write access

@s0 s0 enabled auto-merge October 23, 2022 19:45
@ramprasadagarwal
Copy link
Contributor Author

@s0 Not sure why tests are breaking now, Do you have any suggestions or thoughts?

auto-merge was automatically disabled November 3, 2022 21:33

Head branch was pushed to by a user without write access

@jdcmarques
Copy link
Contributor

@s0 Not sure why tests are breaking now, Do you have any suggestions or thoughts?

Should need #81 to work

@codecov
Copy link

codecov bot commented Nov 17, 2022

Codecov Report

Merging #80 (b3c1c9e) into develop (dd03cd3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #80   +/-   ##
========================================
  Coverage    89.94%   89.94%           
========================================
  Files            1        1           
  Lines          189      189           
  Branches        37       37           
========================================
  Hits           170      170           
  Misses          14       14           
  Partials         5        5           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@s0 s0 enabled auto-merge November 17, 2022 22:10
@s0
Copy link
Owner

s0 commented Nov 17, 2022

Thanks for doing this! :) Also needed to update the package-lock.json, but i added that and rebased, so this should be good to go now

@s0 s0 merged commit cf7d802 into s0:develop Nov 17, 2022
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.

3 participants