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 GH action versions #2010

Closed
idavis opened this issue Nov 6, 2024 · 3 comments · Fixed by #2024
Closed

Update GH action versions #2010

idavis opened this issue Nov 6, 2024 · 3 comments · Fixed by #2024

Comments

@idavis
Copy link
Collaborator

idavis commented Nov 6, 2024

GitHub is about to roll out breaking changes and a few tasks need to be updated.

fuzz.yml

  • update actions/upload-artifact@v3 to actions/upload-artifact@v4

devskim.yml
Aside from that, the codeql action v2 is deprecated and support will end 4 Dec 2024. Need to migrate to v3.

While updating actions, we should also update:

publish-playground.yml

  • update actions/upload-pages-artifact@v1 to actions/upload-pages-artifact@v3
@prkbuilds
Copy link
Contributor

Hi @idavis, I worked something similar for my projects. can I work on this?

@minestarks
Copy link
Member

minestarks commented Nov 7, 2024

@PratikFandade please feel free to take a stab at it. Please beware though, I don't think external contributors can run GitHub actions on this repo without permission from the maintainers, so you may not be able to test your changes without a lot of back and forth... so I'm not sure if this is the most suitable task for an external contributor. But go ahead if you're feeling up for it.

@prkbuilds
Copy link
Contributor

@minestarks Thanks for the heads-up! I understand the limitations with GitHub actions for external contributors. I'll give it a shot and reach out if any issues come up. Appreciate the opportunity to contribute!

@minestarks minestarks linked a pull request Nov 14, 2024 that will close this issue
github-merge-queue bot pushed a commit that referenced this issue Nov 14, 2024
Implemented the changes mentioned in #2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants