Skip to content

Support full downstream skip on LLM branch reject - #72183

Open
guan404ming wants to merge 1 commit into
apache:mainfrom
guan404ming:llm-branch-ignore-trigger-rules
Open

Support full downstream skip on LLM branch reject#72183
guan404ming wants to merge 1 commit into
apache:mainfrom
guan404ming:llm-branch-ignore-trigger-rules

Conversation

@guan404ming

Copy link
Copy Markdown
Member

Why

  • Rejecting a branch review only skipped the direct downstream tasks.
  • A task further down the chain with a permissive trigger rule still ran.

How

  • Add ignore_downstream_trigger_rules to skip every downstream task on reject.
  • Keep the teardown carve-out and the direct-only default unchanged.

This completes the alignment with ApprovalOperator started in #71073.


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Opus 5)

Generated-by: Claude Code (Opus 5) following the guidelines

Rejecting a branch review only skipped the direct downstream tasks, so a
task with a permissive trigger rule further down the chain still ran even
though the reviewer had turned the branch down.
@guan404ming
guan404mingforce-pushed the llm-branch-ignore-trigger-rules branch from 2bcd797 to ddb6b4eCompareAugust 29, 2026 05:40
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@guan404ming