From c920256f2606aae22fba470e4c55aac6d76a6104 Mon Sep 17 00:00:00 2001 From: Mark Harris Date: Wed, 1 Jul 2026 04:33:06 +0000 Subject: [PATCH] CI: don't skip required code-style checks on docs-only PRs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Problem Docs-only PRs (e.g. the CODEOWNERS PR #62) are stuck **BLOCKED**. The `main` ruleset marks the code-style checks (black, include guards, SPDX, whitespace) as **required**, but `codestyle.yml` used workflow-level `paths-ignore` for `**.md`/`CODEOWNERS`. A workflow skipped via `paths-ignore` never reports its checks, so the required checks sit at "Expected" forever and block the merge. ## Fix Remove `paths-ignore` from `codestyle.yml`. The lint jobs are cheap (ubuntu-latest) and now always run and report, satisfying the required checks on every PR including docs-only ones. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.8 Signed-off-by: Mark Harris --- .github/workflows/codestyle.yml | 3 --- 1 file changed, 3 deletions(-) diff --git a/.github/workflows/codestyle.yml b/.github/workflows/codestyle.yml index 924a433..ea3d10e 100644 --- a/.github/workflows/codestyle.yml +++ b/.github/workflows/codestyle.yml @@ -3,9 +3,6 @@ on: pull_request: branches: - main - paths-ignore: - - 'CODEOWNERS' - - '**.md' # Allow subsequent pushes to the same PR or REF to cancel any previous jobs. concurrency: