-
Notifications
You must be signed in to change notification settings - Fork 7
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
require condalock action to run successfully before allowing merge #48
Comments
Thanks @JessicaS11 for opening this issue and adding the note at c65efc7 about checking the Actions tab. I'm hoping that turning the In the meantime, this 'check' will unfortunately need to fall on the Pull Request reviewer 🙂 |
From what I can tell after working a bit on #88, it won't be possible to enable a check under the 'Checks' tab with a slash command like Instead, we would need to use either the |
The condalock action fails silently, so if a test is fixed to pass, the PR checks pass and allow merging even if the conda-lock.yml has not been updated. The
/condalock
action must be re-triggered to run, and should be required to pass before merging is allowed.The text was updated successfully, but these errors were encountered: