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

Upgrade deprecated GitHub Actions in build-git-installers #568

Conversation

dscho
Copy link
Member

@dscho dscho commented Mar 17, 2023

This is a companion to #566.

I pushed a tag to trigger a build that demonstrates that all the deprecation warnings are gone (for now).

EDIT: pointed the link to the build that addresses the download-artifact warnings.

Upgrade 'actions/github-script' from v3 to v6 to silence the Node.js 12
deprecation warning. Note that this also includes an update to how the
'github' object is accessed in the script ('github' -> 'github.rest')
due to a breaking change introduced in v5 [1].

[1] https://github.com/actions/github-script#breaking-changes-in-v5

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Upgrade the 'actions/upload-artifact' Action to v3 to stop using Node.js
12 (deprecated).

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Upgrade the 'actions/upload-artifact' Action to v3 to stop using Node.js
12 (deprecated).

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Upgrade the 'actions/upload-artifact' Action to v3 to stop using Node.js
12 (deprecated).

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
The `ubuntu-18.04` runners are deprecated for a while now [1], and will
go away this year's April Fools Day (sic!).

Let's switch to the next-oldest LTS version of Ubuntu, allowing the
broadest range of Ubuntu users to consume the generated `.deb` package.

[1] actions/runner-images#6002

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho dscho requested a review from vdye March 17, 2023 12:13
@dscho
Copy link
Member Author

dscho commented Mar 17, 2023

I could not test 258464d because that would release a new .deb package via apt. To accommodate for that, I triple-checked that this commit is correct, and would appreciate an extra pair of eyes over the changes.

 Upgrade the 'actions/download-artifact' Action to v3 to stop using
 Node.js 12 (deprecated).

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Upgrade the 'actions/download-artifact' Action to v3 to stop using
Node.js 12 (deprecated).

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Upgrade the 'actions/download-artifact' Action to v3 to stop using
Node.js 12 (deprecated).

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Upgrade the 'actions/download-artifact' Action to v3 to stop using
Node.js 12 (deprecated).

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho
Copy link
Member Author

dscho commented Mar 17, 2023

Oh, I forgot the actions/download-artifact...

@dscho
Copy link
Member Author

dscho commented Mar 17, 2023

Hooray, https://github.com/microsoft/git/actions/runs/4447499276 passed without warnings! And on that note, I'll mark this as ready for review.

@dscho dscho marked this pull request as ready for review March 17, 2023 13:19
Copy link
Collaborator

@derrickstolee derrickstolee left a comment

Choose a reason for hiding this comment

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

@dscho dscho merged commit 4eb9cb3 into microsoft:vfs-2.40.0 Mar 20, 2023
@dscho dscho deleted the upgrade-deprecated-github-actions-in-build-git-installers branch March 20, 2023 12:33
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.

2 participants