Skip to content

Fix benchmark status cancellation race - #27

Merged
ludfjig merged 1 commit into
mainfrom
fix/benchmark-status-concurrency
Sep 24, 2026
Merged

ludfjig merged 1 commit into
mainfrom
fix/benchmark-status-concurrency

Conversation

@ludfjig

@ludfjig ludfjig commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Summary:

  • keep benchmark workload cancellation at the reusable-workflow job boundary
  • create Benchmark Status outside that boundary for every PR event
  • verify both workflow files and reusable job names before publication

Validation:

  • npm run build
  • workflow YAML parsing
  • git diff --check

Tests were not run.

Signed-off-by: Ludvig Liljenberg <4257730+ludfjig@users.noreply.github.com>
@ludfjig ludfjig added bug Something isn't working benchmarks: skip labels Sep 24, 2026
@ludfjig
ludfjig merged commit d1c2d93 into main Sep 24, 2026
25 of 36 checks passed

This branch was successfully deployed

1 active deployment
github-pages — 7e521584 Deployed Sep 24, 2026 by ludfjig via deploy #287
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

benchmarks: skip bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant