Skip to content

Improve workflows#698

Open
hasezoey wants to merge 17 commits intotramhao:masterfrom
hasezoey:cleanupWorkflows
Open

Improve workflows#698
hasezoey wants to merge 17 commits intotramhao:masterfrom
hasezoey:cleanupWorkflows

Conversation

@hasezoey
Copy link
Copy Markdown
Collaborator

@hasezoey hasezoey commented Apr 11, 2026

This PR applies some changes with regards to what was discussed in #695.
In detail:

  • (explicitly) limit permissions available
  • make the release workflow easier to parse at a glance
  • add options to test the release workflow other than actually releasing things
    • workflow_dispatch: manually trigger a run (without releasing), to make sure it actually fully builds and is still valid
    • pull_request: make sure it is valid and working when a pull request changes it
    • schedule: make sure it stays valid and builds correctly (once every week)

Note that i could not make a broader push test, as paths combined with either tags or branches creates a AND condition instead of a OR. (in this case, only triggers a workflow run if a tag is pushed and that tag modified the given files)
Additionally, i could not figure out a good way to merge build and release yet, but this is still a improvement.

hasezoey added 16 commits April 11, 2026 13:43
As they are unused and likely outdated
By setting same features in test & final build
Default is 360 minutes (6 hours), and stuck workflows dont need to run that long
To align with other names and be more concise
As that, together with "tags" is a AND not a OR.
@hasezoey hasezoey requested a review from tramhao April 11, 2026 14:52
@hasezoey
Copy link
Copy Markdown
Collaborator Author

By the way, it is more than a few month again since the last release, and i would say the current state is rather stable, what do you think about releasing another version soon @tramhao ?

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.

1 participant