Skip to content

ci: registration stub for the bench-campaign dispatch workflow - #922

Merged
marwen-abid merged 1 commit into
mainfrom
bench-automation/workflow-stub
Aug 6, 2026
Merged

ci: registration stub for the bench-campaign dispatch workflow#922
marwen-abid merged 1 commit into
mainfrom
bench-automation/workflow-stub

Conversation

@marwen-abid

@marwen-abid marwen-abid commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

What

add a stub workflow to register the bench-campaign dispatch workflow.

Why

workflow_dispatch in gh only lists workflows present on the default branch. This no-op stub registers the bench campaign so it can be dispatched against feature/full-history, where the real workflow lives (#921). Dispatching the stub ref itself fails with a pointer.

workflow_dispatch only lists workflows present on the default branch. This stub registers the bench campaign so it can be dispatched against feature/full-history, where the real workflow lives until that branch merges.
Dispatching the stub ref itself fails with a pointer.
Copilot AI balanced review requested due to automatic review settings August 5, 2026 23:30
@marwen-abid marwen-abid self-assigned this Aug 5, 2026
@marwen-abid

Copy link
Copy Markdown
Contributor Author

@claude[agent] review

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a default-branch registration stub for dispatching the real benchmark campaign workflow from another branch.

Changes:

  • Mirrors the campaign’s dispatch inputs.
  • Adds an intentional failure directing users to the real workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/bench-campaign.yml
@Claude

Claude AI commented Aug 5, 2026

Copy link
Copy Markdown

@claude[agent] review

Reviewed. The reviewer was right that feature/full-history doesn't yet carry the real workflow — it currently lives on bench-automation/campaign-workflow (PR #921, still an open draft). In dd814af I updated the stub's failure message and header comment to point users to that branch/PR now, and note it moves to feature/full-history once #921 merges.

@marwen-abid
marwen-abid force-pushed the bench-automation/workflow-stub branch from dd814af to caa378c Compare August 5, 2026 23:51
@marwen-abid
marwen-abid merged commit 41b2f0f into main Aug 6, 2026
38 of 41 checks passed
@marwen-abid
marwen-abid deleted the bench-automation/workflow-stub branch August 6, 2026 22:46
@marwen-abid marwen-abid linked an issue Aug 7, 2026 that may be closed by this pull request
8 tasks
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.

e2e benchmark campaigns workflow — dispatch, run on EC2, publish results

4 participants