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

Bump actions/artifact to v2 #9

Merged
merged 3 commits into from
Jul 31, 2024
Merged

Bump actions/artifact to v2 #9

merged 3 commits into from
Jul 31, 2024

Conversation

bbhtt
Copy link
Collaborator

@bbhtt bbhtt commented Jul 29, 2024

Closes #8

@bbhtt bbhtt force-pushed the bbhtt/bump-artifact branch 3 times, most recently from a698041 to 5ad6df8 Compare July 29, 2024 17:03
@bbhtt bbhtt force-pushed the bbhtt/bump-artifact branch from 5ad6df8 to 1c44afa Compare July 29, 2024 17:06
@bbhtt bbhtt requested a review from barthalion July 29, 2024 17:14
@barthalion
Copy link
Member

I guess it's fine. Let's make sure PCSX2/Duckstation/OBS aren't broken once it's released.

@bbhtt
Copy link
Collaborator Author

bbhtt commented Jul 30, 2024

Let's make sure PCSX2/Duckstation/OBS aren't broken once it's released.

They don't seem to be using this functionality directly. I tested it nevertheless

https://github.com/bbhtt/duckstation/actions/runs/10164389327/job/28109613238#step:9:3404

bbhtt/duckstation@149c3c4

@barthalion barthalion merged commit e5fa5a8 into master Jul 31, 2024
19 checks passed
@bbhtt bbhtt deleted the bbhtt/bump-artifact branch September 22, 2024 17:50
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.

Deprecation notice: v1, v2, and v3 of the artifact actions
2 participants