Skip to content

Skip LLM review wait for PRs without .lean files - #30

Merged
justincasher merged 1 commit into
mainfrom
justin/llm-review-skip-non-lean-prs
May 10, 2026
Merged

justincasher merged 1 commit into
mainfrom
justin/llm-review-skip-non-lean-prs

Conversation

@justincasher

Copy link
Copy Markdown
Collaborator

Summary

  • Lean Action CI is path-filtered to Lean sources, so PRs touching only Python / markdown never trigger a CI run. The LLM review workflow was waiting the full 30 minutes on every such PR before timing out and skipping (visible in PR Filter candidates table by classifier decisions #29's CI as the Wait for Lean Action CI step looping "No Lean Action CI run for <sha> yet; waiting...").
  • Move the .lean-file check to the top of the job, gate the wait (and its "CI not green" comment) on at least one .lean file in the PR, and post a one-line skip comment for the rest. Lean PRs keep the existing wait + gpt-5.5 review path unchanged.

Lean Action CI is path-filtered to Lean sources, so PRs that only
touch Python or markdown never produce a CI run. The review workflow
was waiting the full 30 minutes on every such PR before timing out
and skipping. Move the .lean-files check to the front, gate the wait
(and its 'CI not green' comment) on at least one .lean file changed,
and post a one-line skip comment for the rest. Lean PRs flow through
the wait + gpt-5.5 review unchanged.
@justincasher
justincasher merged commit a67fab1 into main May 10, 2026
2 of 3 checks passed
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.

1 participant