Skip to content

Clarify that supported PR actions are any valid Github PR actions - #286

Closed
cosmith wants to merge 1 commit into
getsentry:mainfrom
cosmith:patch-1
Closed

Clarify that supported PR actions are any valid Github PR actions#286
cosmith wants to merge 1 commit into
getsentry:mainfrom
cosmith:patch-1

Conversation

@cosmith

@cosmith cosmith commented May 5, 2026

Copy link
Copy Markdown

Hello, thanks for this tool!

While setting it up I had a Codex review incorrectly flag ready_for_review as unsupported. I checked the source and tested it and it works fine, so my proposal is to soften the skill wording a bit to make it clear that any Github PR action from this list is valid.

image

@dcramer

dcramer commented Aug 4, 2026

Copy link
Copy Markdown
Member

thanks!

@sentry-junior

sentry-junior Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Superseded by #464

This fork PR never got GitHub Actions CI (only Vercel auth failure + external checks), and there was nothing to re-run. #464 is the same docs fix on an internal branch so real CI can attach.

@sentry-junior sentry-junior Bot closed this Aug 4, 2026
dcramer added a commit that referenced this pull request Aug 4, 2026
## Summary

Skill docs still listed only `opened` / `synchronize` / `reopened` /
`closed` for `pull_request` actions. Config already accepts arbitrary
action strings (`z.array(z.string())`) and matches with
`includes(context.action)`, so `ready_for_review` and other GitHub PR
webhook actions already work.

## Changes

- Update `skills/warden/references/config-schema.md`
- Update `skills/warden/references/configuration.md` (same stale list)

## Why not #286

Supersedes [#286](#286). That
was the same docs fix from a fork (`cosmith:patch-1`) where no GitHub
Actions workflows ever ran on the head SHA — only Vercel auth failure +
external checks — so there was nothing to re-run. This internal branch
gets real CI.

## Checks

- Docs-only; no code/runtime change
- CI on this PR is the validation

<!-- junior-request-attribution:start -->
Requested by **David Cramer**.
<!-- junior-request-attribution:end -->

<!-- junior-session-footer:start -->
<!-- junior-conversation-id:slack%3AC0ACFA5JBDX%3A1785871445.612169 -->

--

[View Junior
Session](https://junior-prod.sentry.dev/conversations/slack%3AC0ACFA5JBDX%3A1785871445.612169)
[[Sentry]](https://sentry.sentry.io/explore/conversations/slack%3AC0ACFA5JBDX%3A1785871445.612169/?project=4510944073809921)

<!-- junior-session-footer:end -->

Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com>
Co-authored-by: David Cramer <david@sentry.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants