-
Notifications
You must be signed in to change notification settings - Fork 870
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
[Backport 1.3] [Manual backport 1.x][CCI] Replace node
with scripts/use_node
in functional tests
#4010
Conversation
…) (#3970) * Replace node with scripts/use_node (#3719) Co-authored-by: Sergey Myssak <sergey.myssak@gmail.com> Signed-off-by: Andrey Myssak <andreymyssak@gmail.com> * Update CHANGELOG.md (#3719) Co-authored-by: Sergey Myssak <sergey.myssak@gmail.com> Signed-off-by: Andrey Myssak <andreymyssak@gmail.com> --------- Signed-off-by: Andrey Myssak <andreymyssak@gmail.com> Co-authored-by: Sergey Myssak <sergey.myssak@gmail.com> Co-authored-by: Josh Romero <rmerqg@amazon.com> (cherry picked from commit 8f890e9) Co-authored-by: Andrey Myssak <40265277+andreymyssak@users.noreply.github.com> (cherry picked from commit 501ba8a) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Codecov Report
@@ Coverage Diff @@
## 1.3 #4010 +/- ##
=======================================
Coverage 67.49% 67.50%
=======================================
Files 3044 3044
Lines 58692 58692
Branches 8902 8902
=======================================
+ Hits 39617 39619 +2
+ Misses 16926 16925 -1
+ Partials 2149 2148 -1
Flags with carried forward coverage won't be shown. Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just noticed: 1.3
does not have use_node
and nor does 1.x
.
@AMoo-Miki So we should close this PR and also revert #3970 ? |
Yes please. |
Closing the PR, dont see any value in reverting the change in 1.x since thats a archived branch now. |
Backport 501ba8a from #3970.