Skip to content

Allow benchmarks on merged pull requests - #230

Open
ShiboSoftwareDev wants to merge 1 commit into
mainfrom
fix/benchmark-merged-prs
Open

Allow benchmarks on merged pull requests#230
ShiboSoftwareDev wants to merge 1 commit into
mainfrom
fix/benchmark-merged-prs

Conversation

@ShiboSoftwareDev

Copy link
Copy Markdown
Contributor

Summary

  • allow /benchmark comments and manual PR-number runs to target merged pull requests
  • benchmark an open PR at its exact fork head, or a merged PR at its immutable merge commit in the base repository
  • continue rejecting closed, unmerged pull requests and unavailable commit SHAs
  • document both supported PR states and cover them in the trusted-workflow tests

Validation

  • bun test tests/pr-benchmark-authorization.test.ts tests/benchmark-workflow-permissions.test.ts tests/pr-benchmark-report.test.ts — 3 pass, 0 fail
  • NODE_OPTIONS=--max-old-space-size=8192 bun run typecheck — both configurations pass
  • bun run format:check — passes with the repository's existing large-fixture warnings
  • full bun test — 304 pass; 2 existing local capture/CLI failures caused by the installed capacity-autorouter package missing AutoroutingPipelineSolver9_PreloadedTraceGraph

@vercel

vercel Bot commented Sep 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
fanout-solver Ready Ready Preview Sep 11, 2026 9:47pm UTC

Request Review

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