Skip to content

Remove scheduled job#12

Merged
zecakeh merged 2 commits intomainfrom
zecakeh/rm-scheduled-job
May 12, 2025
Merged

Remove scheduled job#12
zecakeh merged 2 commits intomainfrom
zecakeh/rm-scheduled-job

Conversation

@zecakeh
Copy link
Contributor

@zecakeh zecakeh commented May 5, 2025

GitHub disables scheduled jobs if there hasn't been activity on the repository for the last 60 days. This is likely to happen a lot here since we will probably only update it after breaking ruma releases that should happen every 6 months or so.

Copy link
Member

@jplatte jplatte left a comment

Choose a reason for hiding this comment

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

Could merge the two jobs in this workflow into one then. (that doesn't use with.command at all)

zecakeh added 2 commits May 12, 2025 10:18
GitHub disables scheduled jobs if there hasn't been activity on the
repository for the last 60 days. This is likely to happen a lot here
since we will probably only update it after breaking ruma releases that
should happen every 6 months or so, so we remove the schedule job.

Since both jobs are now always running with the same conditions, we can
just merge them.
Some dependencies require that version.
@zecakeh zecakeh force-pushed the zecakeh/rm-scheduled-job branch from 83a9587 to 277b331 Compare May 12, 2025 08:21
@zecakeh
Copy link
Contributor Author

zecakeh commented May 12, 2025

Nice catch, done!

@zecakeh zecakeh merged commit 90ff257 into main May 12, 2025
10 checks passed
@zecakeh zecakeh deleted the zecakeh/rm-scheduled-job branch May 12, 2025 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants