Skip to content

ci: remove synchronize trigger from Slack PR review caller - #1950

Open
jariy17 wants to merge 1 commit into
mainfrom
chore/slack-remove-synchronize-main
Open

ci: remove synchronize trigger from Slack PR review caller#1950
jariy17 wants to merge 1 commit into
mainfrom
chore/slack-remove-synchronize-main

Conversation

@jariy17

@jariy17 jariy17 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Removes the synchronize (new-revision) trigger from the Slack PR review caller. Only review-requested and review-submitted events notify now; no more "branch updated / please re-review" pings.

Caller-only change — the reusable is untouched (its branch_updated path just goes unused).

@jariy17
jariy17 requested a review from a team August 7, 2026 18:02
@github-actions github-actions Bot added size/xs PR size: XS agentcore-harness-reviewing AgentCore Harness review in progress labels Aug 7, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Aug 7, 2026
@github-actions github-actions Bot removed the agentcore-harness-reviewing AgentCore Harness review in progress label Aug 7, 2026
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.26.0.tgz

How to install

gh release download pr-1950-tarball --repo aws/agentcore-cli --pattern "*.tgz" --dir /tmp/pr-tarball
npm install -g /tmp/pr-tarball/aws-agentcore-0.26.0.tgz

@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
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 40.44% 15153 / 37464
🔵 Statements 39.72% 16160 / 40682
🔵 Functions 34.69% 2596 / 7482
🔵 Branches 33.82% 10108 / 29882
Generated in workflow #4322 for commit d6180a8 by the Vitest Coverage Report Action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant