ci: drop redundant tags-ignore glob#5
Merged
Merged
Conversation
With branches: ['**'] defined for the push event, tags are the "undefined Git ref" — per GitHub's filter rules a push event won't run for tags at all, so the dzil release tags (v0.NNN) already never triggered a build. The tags-ignore: ['*'] line was therefore redundant, and worse, it re-enabled tag evaluation while ['*'] doesn't match slashed tags (e.g. releases/1.0), which would then have built. Removing it leaves: branch pushes + PRs build, all tag pushes don't — no glob corner case. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Getty
added a commit
that referenced
this pull request
Jul 2, 2026
An unrecognized subcommand fell through MooX::Cmd dispatch into the default board view and exited 0, so typos like `karr agent-name` looked like success. Leftover bare-word positionals in the root execute now die with "Unknown command: ..." (MooX::Cmd echoes parsed option flags back into args, so dash-prefixed entries are ignored and bare `karr` / `karr --done` still render the board). `karr archive` was also the only id-taking command that exited 0 when an id did not exist (warn + next, while show/move/edit/delete/handoff all die). A missing id in the batch now exits 1 after the loop: existing ids stay archived and reported — partial success committed, overall failure signalled, matching kanban-md's runBatch semantics. Re-archiving an already-archived task remains a successful no-op. Adds t/41-cli-error-exits.t pinning the whole not-found matrix across id-taking commands plus the unknown-command and bare-karr behaviors. Closes karr tasks #5 and #6. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GLsGFe6Vt98jHpCJmBwmMi
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up zu #4.
branches: ['**']macht Tags zum undefined Git ref — laut GitHubs Filter-Regeln läuft einpush-Event dann gar nicht für Tags. Diedzil release-Tags (v0.NNN) haben also ohnehin nie gebaut.tags-ignore: ['*']war damit:branchessowieso nicht;['*']matcht keine Slash-Tags (z.B.releases/1.0), die dann doch gebaut hätten.Entfernt. Verhalten danach: Branch-Pushes + PRs bauen, alle Tag-Pushes bauen nicht — ohne Glob-Sonderfall.
🤖 Generated with Claude Code