-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Notify team via Slack with digest of recent GitHub comments #8707
Merged
mekarpeles
merged 13 commits into
internetarchive:master
from
jimchamp:feature/gh-issue-slackbot
Jan 24, 2024
Merged
Notify team via Slack with digest of recent GitHub comments #8707
mekarpeles
merged 13 commits into
internetarchive:master
from
jimchamp:feature/gh-issue-slackbot
Jan 24, 2024
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
jimchamp
force-pushed
the
feature/gh-issue-slackbot
branch
from
January 11, 2024 22:26
d16906f
to
6df812a
Compare
jimchamp
force-pushed
the
feature/gh-issue-slackbot
branch
3 times, most recently
from
January 11, 2024 22:57
ecaf941
to
a990104
Compare
jimchamp
force-pushed
the
feature/gh-issue-slackbot
branch
from
January 11, 2024 23:12
276e6a9
to
449fc9a
Compare
jimchamp
force-pushed
the
feature/gh-issue-slackbot
branch
from
January 17, 2024 17:58
2511ed0
to
5a2ce14
Compare
jimchamp
force-pushed
the
feature/gh-issue-slackbot
branch
from
January 17, 2024 21:30
a1a6aa4
to
d64a630
Compare
for more information, see https://pre-commit.ci
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.
Addresses #8704
Adds script that publishes digest of recent GitHub issue comments to a given Slack channel.
The digest is a threaded message containing links to the newest comment in each recently updated issue. The digest excludes comments from staff. If a staff member is a lead for any of the issues, they will be at-mentioned in the Slack message.
Edit: Have also added a workflow that will automatically run this job daily.
Note
New repository secret
SLACK_CHANNEL_ABC_TEAM_PLUS
has been created in support of the new workflow. This seems like it could simply be a variable, but we've set Slack channels as secrets for other workflows (cron_watcher
).Technical
This script makes unauthenticated calls to GitHub APIs, and is subject to a rate limit of 60 calls per hour. If we find that this job is failing due to rate limiting, we can modify this script to make authenticated calls.
Testing
See the "Running the script locally" section in the README for local testing instructions.
Note
The new workflow was run manually on my repo. Results can be seen here.
Screenshot
Stakeholders