Skip to content

Route docs-guild SLA escalations to jkodroff - #21970

Merged
CamSoper merged 1 commit into
masterfrom
claude/docs-guild-escalate-jkodroff
Sep 28, 2026
Merged

CamSoper merged 1 commit into
masterfrom
claude/docs-guild-escalate-jkodroff

Conversation

@CamSoper

Copy link
Copy Markdown
Contributor

Proposed changes

Changes the docs-guild escalate_to in .github/review-routing.yml from tatcoo-pulumi to jkodroff, at Cam's request.

That field sets who the SLA sweep @-mentions when a PR waits past docs-guild's 3-business-day review SLA. It's also the owner the weekly digest shows on the docs-guild line (#21952).

The sweep is still off (REVIEW_V3_SLA unset), so nobody gets pinged until it's switched on. No other file references the old handle. The config parses, and the routing and SLA-sweep tests pass (122 tests).

Related issues (optional)

Related: #21952

🤖 Generated with Claude Code

https://claude.ai/code/session_01Gcx8uerksNXPswPTofF6ke


Generated by Claude Code

@github-actions

github-actions Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Sentinel — merge gate status (preview)

Warning

⚠️ Preview mode — this check is NOT blocking your merge

The Sentinel is in preview (report-only) mode: its check-run concludes neutral whatever the gates below say, so nothing here gates this PR. The rows are informational and safe to ignore for now. It would have concluded failure.

This will be enforced in the near future. Once it is, every row below has to be green before this PR can merge. The rows are the sign-offs a merge needs — some yours, some a reviewer's or a deploy's. Tell us in #docs if one looks wrong.

2 of 5 gates would block this merge once the Sentinel is enforced.

Gate What it needs
🔴 G1 review-ran No current review for 743f02b95 — push to refresh, comment @claude #update-review, or flip the PR to draft and back to ready.
➖ G2 findings-answered Doesn't apply to this PR: no review present — see G1.
🔴 G3 right-approver Needs approval from a member of any review team (pulumi/docs-blog-review, pulumi/docs-guild, pulumi/docs-marketing-review, pulumi/docs-tools) or a repository administrator — no qualifying human approval at this head (bot approvals never count).
➖ G4 infra-evidence Doesn't apply to this PR: no changed path affects the deploy.
➖ G5 oversized-ack Doesn't apply to this PR: not oversized.

Break glass: a member of a routing team (.github/review-routing.yml teams:) can apply review:waived, which is logged.

Evaluated at head 743f02b95.

@CamSoper
CamSoper marked this pull request as ready for review September 28, 2026 22:55
@CamSoper
CamSoper merged commit 4a5cf7f into master Sep 28, 2026
15 checks passed
@CamSoper
CamSoper deleted the claude/docs-guild-escalate-jkodroff branch September 28, 2026 22:55
@github-actions github-actions Bot added the review:triaging Claude Triage is currently classifying the PR label Sep 28, 2026
@github-actions github-actions Bot added the domain:other PR touches no domain-specific path; reviewed under shared criteria only label Sep 28, 2026
@pulumi-bot
pulumi-bot requested a review from a team September 28, 2026 22:56
@github-actions github-actions Bot removed the review:triaging Claude Triage is currently classifying the PR label Sep 28, 2026
@pulumi-bot

Copy link
Copy Markdown
Collaborator

This branch had an error being deployed

1 failed deployment
testing — 743f02b9 Deployed Sep 28, 2026 by CamSoper via Do cleanup #14467
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain:other PR touches no domain-specific path; reviewed under shared criteria only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants