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

Use flathub-infra/flatpak-github-actions/ to build weekly flatpak #1187

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

SqAtx
Copy link
Contributor

@SqAtx SqAtx commented Feb 9, 2025

Fixes #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

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
@SqAtx
Copy link
Contributor Author

SqAtx commented Feb 9, 2025

We can keep an eye on the original GHA and go back to it when they release a v7 that works :)

TheEvilSkeleton added a commit to TheEvilSkeleton/Bottles that referenced this pull request Feb 9, 2025
TheEvilSkeleton added a commit to TheEvilSkeleton/Bottles that referenced this pull request Feb 9, 2025
@diegogangl diegogangl added the packaging Flatpak packages (anything else = NOPE.png) label Feb 12, 2025
@diegogangl
Copy link
Contributor

Sounds like a good solution for now, merging!

@diegogangl diegogangl merged commit 8b0e72f into getting-things-gnome:master Feb 12, 2025
1 check passed
@SqAtx SqAtx deleted the fix_weekly_flatpak branch February 16, 2025 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packaging Flatpak packages (anything else = NOPE.png)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Weekly flatpak GHA is now failing
3 participants