Skip to content

chore: махни проверката за трейлъри, правилото остава в AGENTS.md - #297

Merged
todorkolev merged 1 commit into
mainfrom
chore/trailer-guard-undo
Aug 9, 2026
Merged

chore: махни проверката за трейлъри, правилото остава в AGENTS.md#297
todorkolev merged 1 commit into
mainfrom
chore/trailer-guard-undo

Conversation

@todorkolev

Copy link
Copy Markdown
Collaborator

Правилото за Co-Authored-By: стои на един ред в AGENTS.md и в шаблона за PR. Отделна проверка за него не се изплаща.

Защо

  • иска fetch-depth: 0 в задължителната работа check, тоест пълна история при всеки PR на всеки сътрудник;
  • добавя начин check да падне по причина, нямаща нищо общо с прегледания код;
  • щеше да блокира feat(web): add /health endpoint and post-deploy smoke check #118, вече отворен PR на @mhunter02, заради трейлър, който се маха с един ред в squash съобщението при мърдж.

Тежестта пада върху всички, а поводът е конвенция, която се спазва с четене.

Какво остава

Уточнението на самото правило - че забраната е за агенти, а трейлърите с хора не се пипат, понеже те са единственото, което държи заслугата на сътрудника след squash.

Спрямо състоянието отпреди предишния коммит нетото е два реда:

.github/PULL_REQUEST_TEMPLATE.md | 2 +-
AGENTS.md                        | 2 +-

ci.yml и .devcontainer/post-create.sh са върнати байт по байт. Няма висящи препратки към махнатите файлове. scripts/: 89 теста минават.

Правилото е на един ред в AGENTS.md и в шаблона за PR. Отделна проверка за
него не се изплаща: искаше пълна история при всеки PR в задължителната работа
`check`, добавяше начин тя да падне по причина, нямаща общо с прегледания код,
и щеше да блокира вече отворен PR на сътрудник заради трейлър, който се маха
с един ред в squash съобщението.
@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

Test coverage

Workspace Lines Δ Branches Δ Functions Statements
apps/etl 75.43% +1.43pp 63.52% +5.32pp 70.00% 74.11%
apps/web 91.02% +1.32pp 82.50% +0.70pp 91.19% 89.73%
packages/config 92.85% +0.05pp 72.22% +0.02pp 92.85% 89.18%
packages/db 94.54% +0.34pp 79.38% +0.38pp 87.19% 91.55%
packages/ingest 88.17% +2.37pp 82.30% +2.30pp 80.17% 86.36%
packages/shared 95.50% +0.10pp 80.83% +0.83pp 92.30% 89.56%
Total (informational) 91.23% 80.72% 86.95% 89.08%

✅ No workspace dropped below its baseline (tolerance 0.5pp).

📈 Coverage rose by more than 1pp — run node scripts/check-coverage.mjs --update locally and commit coverage-baseline.json to ratchet the threshold up.

@todorkolev
todorkolev merged commit 8689212 into main Aug 9, 2026
5 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