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 v14 #5130

Merged
merged 1 commit into from
Jan 4, 2022
Merged

upgrade node to v14 #5130

merged 1 commit into from
Jan 4, 2022

Conversation

alicewriteswrongs
Copy link
Contributor

@alicewriteswrongs alicewriteswrongs commented Dec 14, 2021

Pre-Flight checklist

  • Testing
    • Code is tested
    • Changes have been manually tested

What are the relevant tickets?

part of #5129

What's this PR do?

This upgrades our Node version to 14.18.2. This isn't fully updated (current version is 17.x.x) but this will get us to the point where we can upgrade yarn to v3 and run some other upgrades that will then, I think, let us upgrade to a more recent nodejs version and so on. We're climbing the ladder here :)

How should this be manually tested?

everything should work!

@codecov-commenter
Copy link

codecov-commenter commented Dec 14, 2021

Codecov Report

Merging #5130 (c063746) into master (aee034e) will decrease coverage by 0.00%.
The diff coverage is n/a.

❗ Current head c063746 differs from pull request most recent head 9f45e86. Consider uploading reports for the commit 9f45e86 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5130      +/-   ##
==========================================
- Coverage   93.94%   93.94%   -0.01%     
==========================================
  Files         498      498              
  Lines       23121    23119       -2     
  Branches     1415     1415              
==========================================
- Hits        21722    21720       -2     
  Misses       1294     1294              
  Partials      105      105              
Impacted Files Coverage Δ
static/js/global_init.js 93.75% <ø> (ø)
micromasters/settings.py 89.55% <0.00%> (-0.06%) ⬇️
ui/views.py 92.30% <0.00%> (ø)
financialaid/views.py 97.20% <0.00%> (ø)
static/js/constants.js 100.00% <0.00%> (ø)

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 aee034e...9f45e86. Read the comment docs.

@alicewriteswrongs alicewriteswrongs mentioned this pull request Dec 14, 2021
3 tasks
@umarmughal824 umarmughal824 self-assigned this Dec 23, 2021
Copy link
Contributor

@umarmughal824 umarmughal824 left a comment

Choose a reason for hiding this comment

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

👍 looks good

This upgrades nodejs to v14.18.2. This This isn't fully updated (current
version is 17.x.x) but this will get us to the point where we can
upgrade yarn to v3 and run some other upgrades that will then, I think,
let us upgrade to a more recent nodejs version and so on. We're climbing
the ladder here :)

part of #5129
pr #5130
@odlbot odlbot had a problem deploying to micromasters-ci-pr-5130 January 4, 2022 14:30 Failure
@alicewriteswrongs alicewriteswrongs merged commit 9f45e86 into master Jan 4, 2022
@alicewriteswrongs alicewriteswrongs deleted the ap/upgrade-node-lint branch January 4, 2022 16:48
@alicewriteswrongs alicewriteswrongs temporarily deployed to micromasters-ci January 4, 2022 17:00 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants