From 66fe7fe402e7b8468cebeb7952e4da98afc3bddf Mon Sep 17 00:00:00 2001 From: jojoprison Date: Sun, 25 Oct 2026 13:29:19 +0700 Subject: [PATCH] =?UTF-8?q?ci:=20=D0=BB=D0=B8=D0=BD=D1=82=D0=B5=D1=80=20?= =?UTF-8?q?=D0=BD=D0=B0=20=D0=BA=D0=B0=D0=B6=D0=B4=D0=BE=D0=BC=20PR=20?= =?UTF-8?q?=E2=80=94=20=D0=B8=D0=BD=D0=B0=D1=87=D0=B5=20PR=20=D1=81=20?= =?UTF-8?q?=D0=BE=D0=B4=D0=BD=D0=B8=D0=BC=D0=B8=20=D0=B4=D0=BE=D0=BA=D0=B0?= =?UTF-8?q?=D0=BC=D0=B8=20=D0=BD=D0=B5=20=D0=B2=D0=BB=D0=B8=D0=B2=D0=B0?= =?UTF-8?q?=D0=B5=D1=82=D1=81=D1=8F?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit lint — обязательная проверка main, а skill-lint.yml запускался на PR только при правках плагина, манифестов и тестов. На остальных PR проверка не стартовала, и защита ветки ждала её вечно. Фильтр у push оставлен. Co-Authored-By: Claude Opus 5.5 --- .github/workflows/skill-lint.yml | 14 +++----------- CHANGELOG.md | 4 ++++ 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/.github/workflows/skill-lint.yml b/.github/workflows/skill-lint.yml index e577e31..32a2722 100644 --- a/.github/workflows/skill-lint.yml +++ b/.github/workflows/skill-lint.yml @@ -14,18 +14,10 @@ on: - "scripts/verify-release.py" - ".github/workflows/skill-lint.yml" - ".github/workflows/release.yml" + # No paths filter on PRs: `lint` is a required check on main, and a filtered + # workflow never reports on a PR outside its paths — a docs-only PR would wait + # for it forever and could never merge. pull_request: - paths: - - "plugins/**" - - ".claude-plugin/marketplace.json" - - ".agents/plugins/marketplace.json" - - "config.example.json" - - "CHANGELOG.md" - - "scripts/lint-skills.py" - - "scripts/test-*.py" - - "scripts/verify-release.py" - - ".github/workflows/skill-lint.yml" - - ".github/workflows/release.yml" jobs: lint: diff --git a/CHANGELOG.md b/CHANGELOG.md index 4826210..52c647f 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -6,6 +6,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/), ## [Unreleased] +### Fixed + +- **A docs-only PR could never merge.** `lint` is a required check on `main`, but `skill-lint.yml` ran on PRs only when they touched the plugin, manifests or tests. On any other PR the check never started, so the branch protection waited for it forever. The workflow now runs on every PR; the `push` trigger keeps its path filter. + ## [1.2.18] - 2026-09-05 ### Fixed