Skip to content

chore: promote auto-classify Cancelled deploy mapping to main - #47

Merged
LukasWodka merged 2 commits into
mainfrom
develop
May 15, 2026
Merged

chore: promote auto-classify Cancelled deploy mapping to main#47
LukasWodka merged 2 commits into
mainfrom
develop

Conversation

@LukasWodka

Copy link
Copy Markdown
Contributor

Carries #46 to main.

LukasWodkaand others added 2 commits May 15, 2026 11:54
Lukas added a "Cancelled" option to the Deploy environment field so the
"Deployment environment" board view can show cancelled items in their
own column instead of bucketing them with not-yet-deployed work in
"none".
Update the cron mapping so Status=Cancelled rows get Deploy=Cancelled
on the next reconcile pass (and stay there even if someone manually
moves them).
Mapping now:
Backlog / Ready / In progress / Code review -> none
FR on dev / Ready for staging -> dev
FR on staging / Ready for prod -> staging
Prod -> prod
Cancelled -> Cancelled
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
fix(auto-classify): map Status=Cancelled → Deploy=Cancelled
@LukasWodka
LukasWodka merged commit 216e4be into mainMay 15, 2026
LukasWodka added a commit that referenced this pull request Aug 6, 2026
Bugbot on the duplicate PR (#181) caught a regression I introduced in the
previous commit, and it would have hurt: PR_NUMBERS is scraped from commit
subjects, so it legitimately carries numbers that are not PRs in this repo --
a hand-written (#47) issue reference, or a subject carried in from another
repo. The loop above already documents exactly that. Failing closed on those
turned ordinary pushes red.
Fail-closed is right for an outage and wrong for a reference that simply does
not resolve. The two are now split on the error text:
* "Could not resolve to a PullRequest" / "...to an Issue" -> notice, skip
* anything else (401, rate limit, permissions) -> warning + RUN_FAILED
2>/dev/null is also gone. It discarded the only evidence that could tell the
two apart, and left any red run undiagnosable -- barely better than a silent
one. The error text now reaches the log.
Verified with a stub: a normal read passes, #47 skips as a notice, a 401 sets
RUN_FAILED.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Sign up for freeto 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

@LukasWodka