Skip to content

Group non-default cargo ci workflow commands - #5664

Merged
bfops merged 8 commits into
masterfrom
bot/cargo-ci-other-workflows
Aug 5, 2026
Merged

Group non-default cargo ci workflow commands#5664
bfops merged 8 commits into
masterfrom
bot/cargo-ci-other-workflows

Conversation

@clockwork-labs-bot

@clockwork-labs-bot clockwork-labs-bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Description of Changes

Moves cargo-ci commands used by workflows outside ci.yml under cargo ci other-workflows:

  • cla-assistant

The default cargo ci no-subcommand path now skips other-workflows directly because those commands are not intended to be run as part of someone verifying the CI locally.

The point of having a single subcommand for these is that it means we don't have to maintain a whitelist/blacklist of what gets run when you just run cargo ci.

API and ABI breaking changes

None.

Expected complexity level and risk

  1. Mechanical CLI grouping and workflow call-site updates.

Testing

  • CI passes

Comment thread tools/ci/src/main.rs Outdated
Comment thread tools/ci/src/main.rs Outdated
bfops and others added 2 commits August 4, 2026 11:38
Signed-off-by: Zeke Foppa <196249+bfops@users.noreply.github.com>
Comment thread tools/ci/src/main.rs Outdated
Comment thread tools/ci/src/main.rs Outdated
Signed-off-by: Zeke Foppa <196249+bfops@users.noreply.github.com>
Comment thread tools/ci/src/main.rs Outdated
@bfops bfops changed the title Group non-ci cargo ci workflow commands Group non-defaulgt cargo ci workflow commands Aug 4, 2026
@bfops bfops changed the title Group non-defaulgt cargo ci workflow commands Group non-default cargo ci workflow commands Aug 4, 2026
Comment thread tools/ci/src/main.rs Outdated
Signed-off-by: Zeke Foppa <196249+bfops@users.noreply.github.com>

@jdetter jdetter left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, not tested manually by me

@bfops
bfops added this pull request to the merge queue Aug 4, 2026
Merged via the queue into master with commit ec82998 Aug 5, 2026
44 checks passed
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.

3 participants