Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Update nodejs to 18 #8514

Merged
merged 7 commits into from
May 31, 2023
Merged

Update nodejs to 18 #8514

merged 7 commits into from
May 31, 2023

Conversation

shuse2
Copy link
Collaborator

@shuse2 shuse2 commented May 30, 2023

What was the problem?

This PR resolves #7984

How was it solved?

  • Update node version to be minimum 18 for all the packages

How was it tested?

N/A

@shuse2 shuse2 requested review from ishantiw and bobanm May 30, 2023 15:37
@shuse2 shuse2 self-assigned this May 30, 2023
@shuse2 shuse2 requested review from mosmartin and removed request for bobanm May 30, 2023 15:37
@codecov
Copy link

codecov bot commented May 31, 2023

Codecov Report

Merging #8514 (585b42a) into release/6.0.0 (82aeb67) will decrease coverage by 0.01%.
The diff coverage is n/a.

❗ Current head 585b42a differs from pull request most recent head 72d16cf. Consider uploading reports for the commit 72d16cf to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                @@
##           release/6.0.0    #8514      +/-   ##
=================================================
- Coverage          83.32%   83.32%   -0.01%     
=================================================
  Files                593      593              
  Lines              22323    22323              
  Branches            3271     3271              
=================================================
- Hits               18601    18600       -1     
- Misses              3722     3723       +1     

see 1 file with indirect coverage changes

@mosmartin
Copy link
Contributor

Hi @shuse2, is it prudent to update the README as well as it references node v16?

@mosmartin mosmartin closed this May 31, 2023
@mosmartin mosmartin reopened this May 31, 2023
@shuse2 shuse2 enabled auto-merge (squash) May 31, 2023 13:42
@shuse2 shuse2 merged commit 0914fe2 into release/6.0.0 May 31, 2023
@shuse2 shuse2 deleted the 7984-update-node branch May 31, 2023 15:40
shuse2 added a commit that referenced this pull request Jun 1, 2023
* ⬆️ Update nodejs to 18

* ♻️ Update node version for action

* 🐛 Update test to support node 18 connection

* ♻️ Fix ws integration test

* ♻️ Update README and missing type version
mitsuaki-u pushed a commit that referenced this pull request Jun 2, 2023
* ⬆️ Update nodejs to 18

* ♻️ Update node version for action

* 🐛 Update test to support node 18 connection

* ♻️ Fix ws integration test

* ♻️ Update README and missing type version
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants