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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(ci): Bump GitHub Action workflows to latest major versions #139

Merged
merged 1 commit into from Mar 13, 2024

Conversation

adamdmharvey
Copy link
Contributor

@adamdmharvey adamdmharvey commented Mar 13, 2024

Some spring cleaning 馃Ч .

I noticed in the CI output that GitHub Actions was warning about an older version of Node v16 being used which is deprecated. I believe this is due to some of the GH-provided actions.

image

This PR bumps the GH provided ones. In particular, to use upload-artifact v3 you must also use deploy v4 based on how the upload/download was changed. There's another breaking change in there re: file modes, but looking at the CI output this workflow is not affected by that.

I've left the pinned Ruby action for now for future maintenance.

This also gets many bug fixes, and suggests the artifact workflow is significantly faster (even though CI runs pretty quickly here anyway).

Signed-off-by: Adam Harvey <33203301+adamdmharvey@users.noreply.github.com>
@adamdmharvey adamdmharvey changed the title chore(ci): Bump GitHub Action workflows to latest majors chore(ci): Bump GitHub Action workflows to latest major versions Mar 13, 2024
@adamdmharvey
Copy link
Contributor Author

adamdmharvey commented Mar 13, 2024

The build workflow went clean. It seems to have removed some of the Node version warnings, so that helps a bit with maintenance. Now just the Ruby one left.

image

@adamdmharvey
Copy link
Contributor Author

For whomever may merge this, a trigger of the deploy should be watched just to make sure it doesn't cause any side effects, but based on my reading of the changelogs for these actions, I believe it should go clean!

@adamdmharvey adamdmharvey marked this pull request as ready for review March 13, 2024 21:13
@BryanQuigley
Copy link
Member

Looks good to me. Opened timwis/jkan#283 to track getting this upstream.

@BryanQuigley BryanQuigley merged commit 9b7080c into opendataphilly:main Mar 13, 2024
3 checks passed
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.

None yet

2 participants