Skip to content

ci: add Slack PR review notification caller - #246

Open
jariy17 wants to merge 1 commit into
mainfrom
feat/slack-pr-review-notification
Open

ci: add Slack PR review notification caller#246
jariy17 wants to merge 1 commit into
mainfrom
feat/slack-pr-review-notification

Conversation

@jariy17

@jariy17 jariy17 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Adds the caller that pings PR reviewers in Slack via the shared reusable reusable-slack-pr-review-notification (pinned @626595f, merged to devtools main).

event pings
review requested the requested reviewer
review submitted the PR author
new revision (synchronize) prior reviewers + commenters + pending requested reviewers

Mentions render as real Slack pills (Person-type vars via SLACK_PR_REVIEW_USER_MAP). Secrets + reader-role grants already in place for this repo. Same caller already merged/open for agentcore-cli.

@jariy17
jariy17 requested a review from a team August 7, 2026 17:36
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code security review in progress label Aug 7, 2026
@agentcore-devx-automation

Copy link
Copy Markdown
Contributor

Claude Security Review: no high-confidence findings. (run)

@agentcore-devx-automation agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code security review in progress label Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant