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

Use newest stable node version #2632

Merged

Conversation

karelhala
Copy link
Contributor

No description provided.

@karelhala karelhala requested a review from a team September 5, 2023 06:19
@karelhala karelhala force-pushed the use-latest-node-stable branch 2 times, most recently from e83effc to f10ebbd Compare September 5, 2023 06:30
.travis.yml Outdated
notifications:
email: false
slack:
secure: 'd3g5oKl5kfzi2L96QTrj2g77lzLWzjnM6Ct2OTXRrk90GxxhwHS8RH4Q1IgBqkQjw68nkq7CNE9oAs+pt3si8kT4GQ164YLoTMVkDPBGOoOoMACPyanxQOEluUfdLEDIzChy/7EdlY3l1J2IM+lOK4e95jUwsT9BSXBsSladP++1EUiZfon9JcQSiSZa0e1/cVxLpEDuBB2cruUSTZ9sUATe/XZ0uH1EGzrooQJMkRgdty5UyNSsMxkvAY0Haivq8u9/gWAsLar0bA/90M5CVK7yohh9fY9UfTUbXVqwR3dFAXuW+SURrFVnPAX4FLZt/D09cg/CVCvoasiZdNi9RAeKOCfN+FoxB2ZJNnuM+4KDJX3dxnatd/stmEH1bcd75i4mh9zOWE1HX5d23HuZ4sKdDPpvhG3l7SpZfhLv0/EKL10ld9RdIaiTO2uPI3rsoyDeArzeV+09+dbB1iPKnS/3/Iw5KLhbew3mdJXKVfRk6KYcJySjT8EltrNy5Y7mty7/JzWssSpIpkMCnu6RGAtXO2v/jUfFm1WvsKK5BeH5efbLi1sjMNbVTeA3Bp8pvRPuw+50l94uDfeQ1HUTrq5zKXMUaG9dTdaYZX4fEDMmrbM1TLudjb9Xj4elaA5ioBH0gRFxAnSVTmrKFtFoCRVaqfmj3ceqxttCLUpkfvI='
node_js:
- '16'
- '20.6.0'
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we use v18? Its the current LTS

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure thing!

@karelhala karelhala force-pushed the use-latest-node-stable branch from f10ebbd to 9fa260c Compare September 5, 2023 10:37
@karelhala karelhala force-pushed the use-latest-node-stable branch from 9fa260c to 4899162 Compare September 5, 2023 10:43
@codecov-commenter
Copy link

codecov-commenter commented Sep 5, 2023

Codecov Report

Merging #2632 (2bc13a2) into master (35e56e6) will increase coverage by 5.28%.
Report is 2 commits behind head on master.
The diff coverage is 0.00%.

❗ Current head 2bc13a2 differs from pull request most recent head 4899162. Consider uploading reports for the commit 4899162 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2632      +/-   ##
==========================================
+ Coverage   52.10%   57.38%   +5.28%     
==========================================
  Files          91       91              
  Lines        2451     2783     +332     
  Branches      447      570     +123     
==========================================
+ Hits         1277     1597     +320     
- Misses        810     1185     +375     
+ Partials      364        1     -363     
Files Changed Coverage
src/utils/iqeEnablement.ts 0.00%

@Hyperkid123
Copy link
Contributor

/retest

1 similar comment
@Hyperkid123
Copy link
Contributor

/retest

@Hyperkid123 Hyperkid123 merged commit 441c3d5 into RedHatInsights:master Sep 5, 2023
6 checks passed
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