-
Notifications
You must be signed in to change notification settings - Fork 37
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
Improve Test Automation #8
Conversation
dcccbcb
to
c670de9
Compare
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.
LGTM, this is very clean! I wanted to verify the manual workflow_dispatch worked, but I couldn't figure out how from the PR. I guess if it's broken we can just have another PR to fix it.
Yeah, as far as i can tell you cannot run the workflow that isn't in main yet (there's an option to run a different version from another branch, but not a new workflow) |
@NathanielMcVicar , so far looks good:
UPDATE: runs looked good through the weekend |
main