FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat: bump to use node20 runtime, actions/checkout to v4 by kbdharun · Pull Request #727 · actions/setup-python · GitHub

feat: bump to use node20 runtime, actions/checkout to v4 - #727

Closed
kbdharun wants to merge 3 commits into
actions:mainfrom
kbdharun:patch-1
Closed

feat: bump to use node20 runtime, actions/checkout to v4#727
kbdharun wants to merge 3 commits into
actions:mainfrom
kbdharun:patch-1

Conversation

kbdharun commented Sep 5, 2023
edited
Loading

Copy link
Copy Markdown

Description:

Node20 was added to Actions Runner on v2.308.0 and Node16 reaches end of life soon on 11 Sep 2023.

Therefore, This PR updates the default runtime to node20. I have bumped the actions/checkout version to v4 for the same.

Related issue:

actions/runner#2732

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

A major version bump might be needed after the PRs merge.
I referred to actions/setup-go#421 for this PR.

kbdharun requested a review from a team as a code owner September 5, 2023 03:28
kbdharun changed the title feat: bump to use node20 runtime feat: bump to use node20 runtime, actions/checkout to v4 Sep 5, 2023

Copy link
Copy Markdown
Contributor

Hello @kbdharun. Thank you for your pull request. Could you please revert changes in docs/adrs/0000-caching-dependencies.md

Copy link
Copy Markdown
Contributor

what is blocking the PR merge?

akx commented Dec 12, 2023

Copy link
Copy Markdown
Contributor

This was superseded by #772.

Copy link
Copy Markdown
Author

This was superseded by #772.

Thanks for the information 😄 , glad it got merged.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

5 participants


Back | FazBrowse Home | New Git URL