Skip to content

Stop restarting benchmarks on PR metadata edits - #49

Open
ludfjig wants to merge 1 commit into
mainfrom
skip-ci-on-pr-edits
Open

ludfjig wants to merge 1 commit into
mainfrom
skip-ci-on-pr-edits

Conversation

@ludfjig

@ludfjig ludfjig commented Nov 11, 2026

Copy link
Copy Markdown
Contributor

Editing a PR description or marking a draft ready restarted all 36 benchmark measurement jobs. Neither changes what eligibility reads, which is the head, the base and the benchmarks: skip label. #46 lost a passing run this way.

  • Benchmark and Benchmark Policy no longer run on edited or ready_for_review.
  • Label changes still restart them, since benchmarks: skip changes what runs.

To test: after the smoke run passes, edit this description. Neither workflow should start.

Benchmark and Benchmark Policy ran on edited and ready_for_review events. Editing a PR description or marking a draft ready restarted all 36 measurement jobs and cancelled any measurement in progress, though neither changes the head, base or skip label that eligibility reads.

Both workflows now run only when a PR opens, reopens, gets new commits or changes labels. A PR retargeted to main runs its checks on its next push.

Signed-off-by: Ludvig Liljenberg <4257730+ludfjig@users.noreply.github.com>

This branch has not been deployed

No deployments
Sign up for free to 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