Add GHA manual execution notice #620
Conversation
README.md
Outdated
| @@ -14,3 +14,7 @@ For help and discussion regarding Protobuf plugins, join us on [Slack](https://b | |||
|
|
|||
| For feature requests, bugs, or technical questions, email us at [[email protected]]([email protected]). | |||
|
|
|||
| ## Manual action caveats | |||
|
|
|||
| When triggering a manual execution of the `fetch versions` workflow, you may want to disable the scheduled execution | |||
There was a problem hiding this comment.
The top-level README is more meant for consumers of the plugins from this repo. We should add this sort of info to CONTRIBUTING.md instead.
.github/workflows/fetch_versions.yml
Outdated
| # Run once a day at 10 AM EST | ||
| - cron: "0 14 * * *" | ||
| # Run once a day at 8 AM EST | ||
| - cron: "0 12 * * *" |
There was a problem hiding this comment.
I preferred this run during work hours in the morning since this is when we usually get to these issues.
There was a problem hiding this comment.
What's the rationale for changing this?
There was a problem hiding this comment.
Ahh, okay. The idea here was that they'd be ready to go when you get in in the morning. So you wouldn't need to wait til the scheduled one or manually trigger
There was a problem hiding this comment.
You should receive notifications in Slack about the presence of the PR to remind you, given that you're being assigned
There was a problem hiding this comment.
Typically this runs and alerts us when we are online, it's a good reminder to deal with this repository. Instead of dealing with the backlog of issues that accumulated.
@pkwarren and I get assigned (I can't speak for Philip) but I preferred this run when we're online instead of earlier in the day.
There was a problem hiding this comment.
For some context, we had a bit of a discussion here #471 it used to run the night before (effectively before we get in), and we were trying out the later time.
pgmitche
changed the title
No description provided.