-
Notifications
You must be signed in to change notification settings - Fork 7
Update Renovate schedule for weekly runs #281
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
base: main
Are you sure you want to change the base?
Conversation
|
@erinecon I think the rtd is broken, can you have a look? Thanks! |
Hi @javierdelapuente , I think the RTD build is broken because we haven't merged #282 yet. The error on RTD says
So the RTD build can be ignored for now, or if you want that build to pass, then you'll have to wait until #282 is merged :/ |
| ], | ||
| "schedule": [ | ||
| "* 0-5 1,15 * *" | ||
| "* * * * 0,6" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@javierdelapuente what are you aiming for? I read this as "run every minute from every hours on saturday and sunday".
https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax#onschedule
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Renovate defines that the minute must be * as it cannot be specified at minute level.
It is a copy paste from synapse.
Test results for commit 2fd0b98Test coverage for 2fd0b98 Static code analysis report |
Test results for commit 2fd0b98Test coverage for 2fd0b98 Static code analysis report |
Test results for commit 2fd0b98Test coverage for 2fd0b98 Static code analysis report |
Applicable spec:
Overview
Configure Renovate to run weekly on weekends, rather than fortnightly.
Rationale
Juju Events Changes
Module Changes
Library Changes
Checklist
docs/release-notes/artifacts. If this PR does not require a change artifact, the PR has been tagged withno-release-note.urgent,trivial,complex)