Skip to content

workflows: accept both autosolve and c-autosolve labels#166167

Merged
trunk-io[bot] merged 2 commits intocockroachdb:masterfrom
ajstorm:ajstorm/autosolve-accept-both-labels
Mar 20, 2026
Merged

workflows: accept both autosolve and c-autosolve labels#166167
trunk-io[bot] merged 2 commits intocockroachdb:masterfrom
ajstorm:ajstorm/autosolve-accept-both-labels

Conversation

@ajstorm
Copy link
Collaborator

@ajstorm ajstorm commented Mar 19, 2026

Accept both autosolve and c-autosolve labels as triggers for the
issue auto-solver workflow. Labels are now preserved on the issue after
the workflow completes for tracking purposes.

Release note: None

@trunk-io
Copy link
Contributor

trunk-io bot commented Mar 19, 2026

😎 Merged successfully - details.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

Accept both `autosolve` and `c-autosolve` labels as triggers for the
issue auto-solver workflow. The label removal step dynamically removes
whichever label was applied using the trigger event's label name.

Epic: none

Release note: None

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ajstorm ajstorm force-pushed the ajstorm/autosolve-accept-both-labels branch from 7eb5829 to 1f1699a Compare March 19, 2026 14:14
@ajstorm ajstorm requested a review from dt March 19, 2026 14:16
@ajstorm ajstorm marked this pull request as ready for review March 19, 2026 14:17
@ajstorm ajstorm requested a review from a team as a code owner March 19, 2026 14:17
Remove the step that strips the trigger label from the issue after the
autosolve workflow runs. This preserves both `autosolve` and
`c-autosolve` labels on the issue for tracking purposes.

Epic: none

Release note: None

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ajstorm ajstorm requested a review from rafiss March 19, 2026 14:43
runs-on: [self-hosted, ubuntu_2404]
timeout-minutes: 180
if: github.event.label.name == 'c-autosolve'
if: github.event.label.name == 'autosolve' || github.event.label.name == 'c-autosolve'
Copy link
Collaborator

Choose a reason for hiding this comment

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

TIL labels are not case-senstive.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

You and Claude Code. I have to tell Claude this periodically because it forgets and writes case insensitive code.

@ajstorm ajstorm force-pushed the ajstorm/autosolve-accept-both-labels branch from 0c2a828 to d574355 Compare March 19, 2026 19:46
@trunk-io trunk-io bot merged commit 829129f into cockroachdb:master Mar 20, 2026
52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants