Allow issue_comment_bot.py to run without updating issues or publishing Slack digest
#9142
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Slack token and channel are now optional arguments. If either are missing, the digest will not be published to Slack.
When the `--verbose` flag is passed, detailed information about each issue is printed.
|
LGTM, please merge when the merge check passes (which seems stalled) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #9143
AddsIssues which require feedback from lead
labels are not added to the issues.
--verboseand--no-labelsflags, and makes the Slack token and channel optional arguments. Digest will only be published to Slack of both the token and the channel are present when the script is called. Ournew_comment_digestaction has been updated to accommodate these changes.If the
--no-labelsflag is included, Needs: ResponseDetailed information about each identified issue is printed to
stdoutwhenever the--verboseflag is included.Technical
Testing
Screenshot
Stakeholders