feat: add skip-actors bypass for bot PRs (#32) - #33
Merged
Conversation
oto-macenauer-absa
requested review from
miroslavpojer and
tmikula-dev
as code owners
July 15, 2026 11:50
oto-macenauer-absa
force-pushed
the
feature/32-skip-actors-bypass
branch
from
July 15, 2026 11:58
b10fb68 to
53f887c
Compare
feat: add skip-actors and skip-labels bypass for bot/exempt PRs (#32) Dependabot-opened PRs fail the default checks: they never link an issue, and their default title (Bump X from Y to Z) is not a conventional commit. Add two bypass inputs that skip all checks and report a pass: - skip-actors: comma-separated PR-author logins (default dependabot[bot]); needs pr-author wired. - skip-labels: comma-separated PR labels (default empty); needs labels wired. check.sh short-circuits to a pass on either match; the release-notes step is gated off via the Detect bypass step, which evaluates both. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> @
oto-macenauer-absa
force-pushed
the
feature/32-skip-actors-bypass
branch
from
July 15, 2026 12:07
53f887c to
ecc5fe3
Compare
tmikula-dev
reviewed
Jul 29, 2026
tmikula-dev
left a comment
tmikula-dev
left a comment
Collaborator
There was a problem hiding this comment.
Interesting to see tool written in the shell. Python would make control flow of this whole tool IMO "cleaner" and easier to test than bash. That is why the changes looked "weird" in the first place. However, understand bash's advantage that it runs natively on every GitHub Actions runner with zero setup.
Please see my comments.
- Default `skip-actors` to empty so both bypass inputs are opt-in and consistent; `dependabot[bot]` is now shown only in the examples/docs. - Wire `labels` and add a `skip-labels` example to examples/absa_aligned.yml. - Drop issue numbers from test section comments. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XhWUoyUSda5v2abkVMdTCg
tmikula-dev
approved these changes
Aug 4, 2026
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
2 participants
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.
@
Closes #32.
Problem
Dependabot-opened PRs fail the action under default config:
Bump X from Y to Zhas notype:prefix.Change
Two bypass inputs that skip all checks and report a pass:
pr-author+skip-actors— comma-separated author logins, defaultdependabot[bot].labels+skip-labels— comma-separated PR labels, default empty (opt-in), e.g.skip-checks.check.shshort-circuits to a pass on either match (writes a "skipped" summary + outputs, exits 0). The release-notes step is gated off via aDetect bypasscomposite step that evaluates both actor and label rules.Tests
Orchestrator cases for both dimensions: match bypasses a failing PR, non-match still fails, empty value disables the bypass, CSV whitespace tolerated. Full suite passes (22/22 orchestrator).
Docs
README input tables + example workflows updated (
check_pr_requirements.ymlwireslabelsviajoin(...labels.*.name, ",");absa_aligned.ymlgets the actor bypass).🤖 Generated with Claude Code
@