-
Notifications
You must be signed in to change notification settings - Fork 38
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
add workflow_dispatch to nightly integ tests #50
Conversation
This lets us manually trigger them, if we want a run earlier than nightly.
|
@@ -3,6 +3,7 @@ on: | |||
# 7am Mon - Fri, UTC: https://crontab.guru/#0_7_*_*_1-5 | |||
# This corresponds to 2-3am us/eastern (depending on daylight savings), or 11pm - midnight pacific | |||
- cron: '0 7 * * 1-5' | |||
workflow_dispatch: null |
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.
havent done this w/ null before
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.
Is this for allowing manual runs? I'm blind, you said this in the comments lol. I'm assuming this does the same as on: workflow_dispatch
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.
yup!
This lets us manually trigger them, if we want a run earlier than nightly. Sorry for the double-PR (with #45), I forgot this use case. :-)
Standard checks