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

[Backport 2.7] Fix Node.js download link (#4556) #4594

Closed
wants to merge 1 commit into from

Conversation

kavilla
Copy link
Member

@kavilla kavilla commented Jul 19, 2023

Manual backport of d746595 from #4556

https://mirrors.nodejs.org/ is no longer available which prevents the ability from release builds
being made. Restoring back the original link.

Follow-up should be considered about a caching mechanism and param to the CLI to utilize another link or path.

Signed-off-by: Miki <miki@amazon.com>
@kavilla kavilla force-pushed the kavilla/2.7/nodelink branch from 1268126 to dc4c853 Compare July 19, 2023 21:46
@kavilla kavilla added the Skip-Changelog PRs that are too trivial to warrant a changelog or release notes entry label Jul 19, 2023
@codecov
Copy link

codecov bot commented Jul 19, 2023

Codecov Report

Merging #4594 (dc4c853) into 2.7 (a998f8d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              2.7    #4594   +/-   ##
=======================================
  Coverage   66.44%   66.44%           
=======================================
  Files        3229     3229           
  Lines       62045    62045           
  Branches     9597     9597           
=======================================
+ Hits        41226    41228    +2     
+ Misses      18517    18516    -1     
+ Partials     2302     2301    -1     
Flag Coverage Δ
Linux 66.39% <100.00%> (+<0.01%) ⬆️
Windows 66.40% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
src/dev/build/tasks/nodejs/node_download_info.ts 88.88% <100.00%> (ø)

... and 1 file with indirect coverage changes

Copy link
Member

@zhongnansu zhongnansu left a comment

Choose a reason for hiding this comment

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

thx!

Copy link
Member

@ashwin-pc ashwin-pc left a comment

Choose a reason for hiding this comment

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

Why are we backporting it to 2.7?

@ashwin-pc
Copy link
Member

Also cypress tests are failing here

@joshuarrrr
Copy link
Member

@kavilla Do we need to backport to 2.7? If so, do you mind looking at the test failures?

@joshuarrrr joshuarrrr added not in release PRs backported to already shipped releases with no future release planned needs more info Requires more information from poster labels Jul 31, 2023
@AMoo-Miki
Copy link
Collaborator

We have passed the need for this with 2.8 and 2.9 releases.

@AMoo-Miki AMoo-Miki closed this Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs more info Requires more information from poster not in release PRs backported to already shipped releases with no future release planned Skip-Changelog PRs that are too trivial to warrant a changelog or release notes entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants