Skip to content
This repository has been archived by the owner on Sep 8, 2024. It is now read-only.

General project updates #105

Merged
merged 5 commits into from
Sep 26, 2021
Merged

General project updates #105

merged 5 commits into from
Sep 26, 2021

Conversation

naipotato
Copy link
Owner

@naipotato naipotato commented Sep 26, 2021

  • Docker and Fedora workflows deleted from the CI
  • Changed manifest format to JSON
  • Adopted Bilal Elmoussaoui's GitHub Action
  • Updated app-id and URLs reflecting the new repository location
  • Enabled aarch64 builds

The new direction I see for this application is Flatpak only at the moment, so the dedicated workflow for Fedora will not be needed
It already accomplishes almost everything that my manual workflow did. The only missing point is the .Locale runtime.
Builder is not going to gain support for YAML manifests anytime soon, and running the command to generate the JSON version is an easy thing to forget. Better to cut corners and use JSONs directly.
This Dockerfile was from when we used our own docker image for the CI. Since we now use Bilal's, it is no longer needed.
The namespace for this project is now ReplayDev, and since I haven't published it on Flathub yet, I can take advantage of it and update the app-id everywhere
Copy link

@Miqueas Miqueas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it's ok, except that thing about dnf

.github/workflows/flatpak.yml Show resolved Hide resolved
@naipotato naipotato enabled auto-merge (rebase) September 26, 2021 15:44
Copy link

@Miqueas Miqueas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All ok 👍

@naipotato naipotato merged commit 1752318 into main Sep 26, 2021
@naipotato naipotato deleted the nahuelwexd/update-things branch September 26, 2021 15:47
@JulianGro JulianGro mentioned this pull request Feb 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants