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

Update github actions to resolve most node deprecation warnings #27324

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

bdach
Copy link
Collaborator

@bdach bdach commented Feb 22, 2024

As is github tradition, workflows started yelling about running on a node version that was getting sunset, so here we go again.

Relevant bumps:

Notably, actions/upload-artifact is not bumped to v4, although it should be to resolve the node deprecation warnings, because it has more breaking changes and bumping would break dorny/test-reporter (see dorny/test-reporter#363).

Only tested basic CI workflow. Evidence it still works here.

As is github tradition, workflows started yelling about running on a
node version that was getting sunset, so here we go again.

Relevant bumps:

- https://github.com/actions/checkout/releases/tag/v4.0.0
- https://github.com/actions/setup-dotnet/releases/tag/v4.0.0
- https://github.com/actions/cache/releases/tag/v4.0.0
- https://github.com/actions/setup-java/releases/tag/v4.0.0
- https://github.com/peter-evans/create-pull-request/releases/tag/v6.0.0
- https://github.com/dorny/test-reporter/releases/tag/v1.8.0

Notably, `actions/upload-artifact` is _not_ bumped to v4, although it
should be to resolve the node deprecation warnings, because it has more
breaking changes and bumping would break `dorny/test-reporter`
(see dorny/test-reporter#363).
@smoogipoo smoogipoo merged commit 083892e into ppy:master Feb 22, 2024
15 of 17 checks passed
@bdach bdach deleted the github-actions-bump branch February 22, 2024 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants