Skip to content

ci: remove the benchmark pull request title scope - #249

Merged
james-rl merged 1 commit into
mainfrom
james/remove-benchmark-scope
Sep 28, 2026
Merged

james-rl merged 1 commit into
mainfrom
james/remove-benchmark-scope

Conversation

@james-rl

Copy link
Copy Markdown
Contributor

Description

The CLI's benchmark and scenario commands were removed in #248, but the pull request title check still accepted benchmark as a scope. This removes it, so new titles can no longer use a scope for a feature that no longer exists.

Type of Change

  • Code refactoring

Changes Made

  • .github/workflows/pr-title.yml: remove benchmark from the allowed scopes.

Testing

A one-word change to the allowed-scope list, with no other logic touched. The workflow runs on this PR's own title.

Additional Notes

CHANGELOG.md still lists historical benchmark releases. I left it alone because it records past releases.

🤖 Generated with Claude Code

The benchmark commands were removed, so benchmark is no longer a valid
area of the CLI for commit and pull request scopes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@reflex-loop

reflex-loop Bot commented Sep 28, 2026

Copy link
Copy Markdown

🔄 Reflex agent status: Working

Agent Status
agt_5P9vO2IXAzrUHqvi2KtDHV 🔄 Working

This PR was created by Reflex.

This comment updates in place as agents work.

@james-rl
james-rl marked this pull request as ready for review September 28, 2026 18:39
@james-rl
james-rl requested a review from sid-rl September 28, 2026 18:40
@james-rl
james-rl merged commit b585d86 into main Sep 28, 2026
16 checks passed
@james-rl
james-rl deleted the james/remove-benchmark-scope branch September 28, 2026 18:41
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.

2 participants