Skip to content

update build.yaml#2275

Merged
bitcraft merged 2 commits intoTuxemon:developmentfrom
JaskRendix:publish
Mar 17, 2024
Merged

update build.yaml#2275
bitcraft merged 2 commits intoTuxemon:developmentfrom
JaskRendix:publish

Conversation

@JaskRendix
Copy link
Collaborator

@JaskRendix JaskRendix commented Feb 19, 2024

PR fixes the publishing action by going back to the version 3, at least until flatpak-github-actions/flatpak-builder updates to v4

thanks @Murmele I didn't know about the verbose: true

@JaskRendix JaskRendix changed the title test build test build.yaml Feb 19, 2024
@Murmele
Copy link
Contributor

Murmele commented Feb 19, 2024

grafik

@Murmele
Copy link
Contributor

Murmele commented Feb 19, 2024

Don't understand why it doesn't download the flatpak artifact

@JaskRendix
Copy link
Collaborator Author

Don't understand why it doesn't download the flatpak artifact

I try to redo this by going back to the actions/download-artifact@v3

@JaskRendix
Copy link
Collaborator Author

Ja, this is a problem with v4, but v3 works

@Murmele
Copy link
Contributor

Murmele commented Feb 19, 2024

Ah yes it worked! Strange that the version matters here

Breaking Changes

On self hosted runners, additional [firewall rules](https://github.com/actions/toolkit/tree/main/packages/artifact#breaking-changes) may be required.
Downloading artifacts that were created from action/upload-artifact@v3 and below are not supported.

For assistance with breaking changes, see MIGRATION.md.

So as conclusion. The upload-artifact must stick to version 3 until flatpak-github-actions/flatpak-builder updates to v4

@JaskRendix
Copy link
Collaborator Author

JaskRendix commented Feb 19, 2024

I'm going to fix it.

Co-authored-by: Murmele <Murmele@users.noreply.github.com>
@JaskRendix JaskRendix changed the title test build.yaml update build.yaml Feb 19, 2024
@JaskRendix JaskRendix marked this pull request as ready for review February 19, 2024 16:33
@bitcraft
Copy link
Collaborator

@Murmele please take a look when you are able to, thank you!

@Murmele
Copy link
Contributor

Murmele commented Feb 28, 2024

@bitcraft looks good to me. @JaskRendix Thanks for fixing!

@bitcraft bitcraft merged commit 325b64e into Tuxemon:development Mar 17, 2024
@JaskRendix JaskRendix deleted the publish branch March 17, 2024 15:23
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.

3 participants