Skip to content

[LOG-5295] ci: stop [skip ci] suppressing required checks on dependabot lockfile regen - #102

Merged
stevekrenzel merged 2 commits into
mainfrom
fix/dependabot-regen-skip-ci
Aug 12, 2026
Merged

[LOG-5295] ci: stop [skip ci] suppressing required checks on dependabot lockfile regen#102
stevekrenzel merged 2 commits into
mainfrom
fix/dependabot-regen-skip-ci

Conversation

@stevekrenzel

@stevekrenzel stevekrenzel commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Dependabot only manages `package.json`, so a companion workflow regenerates `bun.lock`. That regen commit carried `[skip ci]`, which suppresses all workflows for the commit — including the branch-protection required checks (`check`, `site-check`). So the required checks never re-ran against the fixed lockfile and Dependabot PRs sat BLOCKED even once the lockfile was correct.

Fix: drop `[skip ci]` from the regen commit message so CI runs on the regenerated head and the required checks report green there.

No loop risk: commits pushed with `GITHUB_TOKEN` do not trigger new workflow runs, and the job is already gated on `github.actor == 'dependabot[bot]'`.

@vercel

vercel Bot commented Jul 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
crew Ready Ready Preview Aug 12, 2026 9:00pm

Request Review

[skip ci] suppressed the required check/site-check runs, leaving dependabot PRs BLOCKED even after the lockfile was regenerated.
@stevekrenzel
stevekrenzel force-pushed the fix/dependabot-regen-skip-ci branch from 5bd911b to 3218f7f Compare July 15, 2026 08:19
@stevekrenzel stevekrenzel changed the title ci: stop [skip ci] suppressing required checks on dependabot lockfile regen [LOG-5295] ci: stop [skip ci] suppressing required checks on dependabot lockfile regen Jul 15, 2026

@jgarms jgarms left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@stevekrenzel
stevekrenzel requested a review from a team August 12, 2026 21:22
@stevekrenzel
stevekrenzel merged commit 6428d43 into main Aug 12, 2026
12 of 13 checks passed
@stevekrenzel
stevekrenzel deleted the fix/dependabot-regen-skip-ci branch August 12, 2026 21:23
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