Skip to content

fix(NOJIRA): skip go linter for dependabot PRs - #281

Merged
gomezjdaniel merged 2 commits into
mainfrom
chore/skip-lint-dependabot
Nov 5, 2026
Merged

gomezjdaniel merged 2 commits into
mainfrom
chore/skip-lint-dependabot

Conversation

@gomezjdaniel

@gomezjdaniel gomezjdaniel commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Why

Since #277 the linter runs for bot PRs. On Dependabot PRs the standard-lint check fails even when golangci-lint reports 0 issues. The Publish lint comment step gets a 403 (Resource not accessible by integration) because Dependabot-triggered runs get a read-only GITHUB_TOKEN. Examples: Typeform/responses#739, Typeform/enrichment#247.

What

Skip the lint job when github.actor == 'dependabot[bot]' (see restrictions when Dependabot triggers events).

  • Other bots (e.g. coding agents) are still linted, so this keeps the intent of fix(NOJIRA-123): run linter also for bots #277.
  • If a person pushes a commit to a Dependabot branch, the actor is that person and lint runs as usual.
  • I used the actor instead of a dependabot/ branch-prefix check because anyone can create a branch with that prefix.
  • A skipped job counts as passing for required status checks.

After merge

Run the release workflow so the v1 tag points to this change.

🤖 Generated with Claude Code

Dependabot-triggered runs get a read-only GITHUB_TOKEN, so the
'Publish lint comment' step fails with 403 and marks the check as
failed even when golangci-lint finds no issues. Other bots are still
linted (see #277).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@gomezjdaniel
gomezjdaniel requested a review from a team as a code owner November 5, 2026 13:23
@pr-auditor

pr-auditor Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

✅ Security Analysis Results

No security issues found. 1 files reviewed.


@pr-auditor rescan to re-run · Powered by Claude Sonnet 5.5 · Docs · #security-engineering-team

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@gomezjdaniel
gomezjdaniel merged commit 9f9549b into main Nov 5, 2026
5 of 6 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.

2 participants