-
Notifications
You must be signed in to change notification settings - Fork 92
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
Automatically add comment to PR that requires maintainer to review and rerun integration tests #824
Comments
10 tasks
chuckwondo
added a commit
to chuckwondo/earthaccess
that referenced
this issue
Oct 6, 2024
chuckwondo
added a commit
to chuckwondo/earthaccess
that referenced
this issue
Oct 6, 2024
9 tasks
Is this fixed? If I understand correctly, the fix ought to have resulted in a bot comment on #909, right? |
This isn't working as expected. The bot comments are going in the wrong PR! #922 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As a follow-on to Allow maintainers to run integration tests on PR from a fork, it would be very convenient to enhance the GitHub integration-tests workflow to post a comment to the PR when it requires a maintainer to perform a security review and re-run the tests, and the comment should include a link to the GitHub page for the failed integration tests build, where the maintainer can then click the "Rerun" button.
While a maintainer can currently see failed builds and navigate to them to hit the Rerun button, adding a comment to the PR would make the needed action visible in the PR itself.
The text was updated successfully, but these errors were encountered: