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 artifacts action to version 4.x.x #198

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Murmele
Copy link

@Murmele Murmele commented Sep 11, 2024

Fixes #184

actions/artifact 2 contains upload-artifacts version 4

@Murmele Murmele closed this Sep 11, 2024
@Murmele Murmele reopened this Sep 11, 2024
@Murmele
Copy link
Author

Murmele commented Sep 11, 2024

Fixes CI problems of #196

@Murmele Murmele closed this Sep 11, 2024
@Murmele Murmele reopened this Sep 11, 2024
@flxzt
Copy link

flxzt commented Nov 21, 2024

There is some urgency to this issue : upload-artifact v3 will not work anymore after December 5th. (https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/)
@bilelmoussaoui

SqAtx added a commit to SqAtx/gtg that referenced this pull request Feb 9, 2025
Fixes getting-things-gnome#1184

Our current `flatpak/flatpak-github-actions` is not longer working,
because it is using a deprecated version of the `upload-artifact` GHA.
From
https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/:

> Starting January 30th, 2025, GitHub Actions customers will no longer
> be able to use v3 of https://github.com/actions/upload-artifact or
> https://github.com/actions/download-artifact.

We're using upload-artifact from flatpak-github-actions, and the issue
has been filed there as
flatpak/flatpak-github-actions#214.
flatpak/flatpak-github-actions#184 is expected
to fix that, but the associated PR
flatpak/flatpak-github-actions#198 hasn't been
merged.

The GHA has been forked at
https://github.com/flathub-infra/flatpak-github-actions, and the issue
was fixed there back in July:
flathub-infra/flatpak-github-actions@efb7f71

So let's switch to that. I don't like that it's not versioned and we
need to use the master branch, which may add breaking changes at any
time. But it's better than our current solution, which can no longer
work.

Tested on a fork, and the artifact is created correctly.
SqAtx added a commit to SqAtx/gtg that referenced this pull request Feb 9, 2025
Fixes getting-things-gnome#1184

Our current `flatpak/flatpak-github-actions` is not longer working,
because it is using a deprecated version of the `upload-artifact` GHA.
From
https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/:

> Starting January 30th, 2025, GitHub Actions customers will no longer
> be able to use v3 of https://github.com/actions/upload-artifact or
> https://github.com/actions/download-artifact.

We're using upload-artifact from flatpak-github-actions, and the issue
has been filed there as
flatpak/flatpak-github-actions#214.
flatpak/flatpak-github-actions#184 is expected
to fix that, but the associated PR
flatpak/flatpak-github-actions#198 hasn't been
merged.

The GHA has been forked at
https://github.com/flathub-infra/flatpak-github-actions, and the issue
was fixed there back in July:
flathub-infra/flatpak-github-actions@efb7f71

So let's switch to that. I don't like that it's not versioned and we
need to use the master branch, which may add breaking changes at any
time. But it's better than our current solution, which can no longer
work.

Tested on a fork, and the artifact is created correctly:
https://github.com/SqAtx/gtg/actions/runs/13222384044
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.

Migration to upload-artifact version 4
3 participants