Skip to content

Filter web-flow from the contribution author set - #2

Merged
benjick merged 1 commit into
mainfrom
filter-web-flow
Apr 20, 2026
Merged

Filter web-flow from the contribution author set#2
benjick merged 1 commit into
mainfrom
filter-web-flow

Conversation

@benjick

@benjick benjick commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Commits made or merged through GitHub's web UI list web-flow as committer. It's a regular User (not type: Bot) and doesn't carry the [bot] suffix, so our bot filter missed it and the gate flagged it as unsigned.
  • Added GITHUB_SERVICE_LOGINS set containing web-flow and extended isBotLogin to consult it.
  • New test: contribution filters web-flow (GitHub UI commits) from the author set.

Test plan

  • deno task test:action passes (27/27)
  • Sigil action on this PR no longer lists @web-flow in the unsigned set

🤖 Generated with Claude Code

Commits made or merged through the GitHub web UI list web-flow as
committer. It's a regular User (not type: Bot) so the [bot] suffix
filter missed it, and the gate flagged it as unsigned.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Hey @benjick, thanks for opening this.

Before this can merge, the following needs to be addressed:

The following author needs to sign the CLA:

Once signed, rebase this PR onto the latest main and the CLA check will re-run.

@benjick
benjick merged commit 5dd3647 into main Apr 20, 2026
2 of 3 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.

1 participant