SRE-720: Simplify PR-title preflight — drop two-step skip logic#66
Conversation
Folds the bot-author check into the job-level `if:` and removes the separate "Determine whether to skip" step plus the unused `ignored-actors` workflow input. One step (validate), one condition, same behavior for human + bot authors. If a future case needs to exempt a human account, it's a trivial `&&` on the `if:` to add it back.
PR SummaryLow Risk Overview The bot exemption is now enforced directly in the job-level Reviewed by Cursor Bugbot for commit c1d7116. Bugbot is set up for automated code reviews on this repo. Configure here. |
🤖 Augment PR SummarySummary: This PR simplifies the PR-title preflight workflow by removing the two-step skip mechanism. Changes:
Technical Notes: Behavior remains the same for bot-authored PRs (workflow job is skipped), while human-authored PRs continue to be validated for a Linear issue ID in the title prefix. 🤖 Was this summary useful? React with 👍 or 👎 |
Summary
if::Determine whether to skipstep and the unusedignored-actorsworkflow input.Same behavior for both human and bot authors; just less moving parts. The
ignored-actorsinput was added speculatively in SRE-715 and no consumer uses it — trivial to re-add as another&&clause if a future case needs to exempt a human account.Related: SRE-720, follow-up to SRE-715.
Test plan
SRE-720in title).hash-worker[bot]/dependabot[bot]PR still skips correctly.