Run integration test PR checks nightly#2222
Conversation
|
Oops. Still in draft. Code looks good. Let's |
if everything passes.
I still haven't configured a notification to our team if any of these fail (see backlinked issue). I can do that in a follow-up if we want to start running immediately though. |
|
Huh... workflow files don't seem to be working with an error message: I'll look into it but nothing comes top of mind as EDIT: I think I see the syntax error, will fix |
f4013bf to
1932e6e
Compare
Scheduled for 5AM UTC. This will assist in catching bugs introduced with new runner images.
1932e6e to
9a9987f
Compare
For consistency with our non-autogenerated files and GitHub documentation, we should further indent items in lists (`sequence`).
|
PR check failures are now due to upstream (usually transient) |
aeisenberg
left a comment
There was a problem hiding this comment.
Looks reasonable. If the failures are transient, I'll leave it up to you if you want to override the failures and merge or re-run later.
|
I don't think we have a way to force green on this repo, and I'd rather not make all those checks non-required (and then make them required again later), so I'll wait to re-run. |
Scheduled for 5AM UTC. This will assist in catching bugs introduced with new runner images.
Merge / deployment checklist