senokosov/pre-commit-hooks
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

feat: перевести 4 запрета-хука в fail-режим #2

Closed
claude-secretary wants to merge 0 commits from feature/fail-mode-warn-hooks into main
pull from: feature/fail-mode-warn-hooks
merge into: senokosov:main
Conversation 1 Commits - Files changed -
claude-secretary commented 2026-06-09 23:07:04 +07:00
Owner
Copy link

Что

no-underscore-filenames, no-private-imports, no-private-method-calls, no-issue-refs-in-comments переведены из warn-only в fail: sys.exit(0) → exit 1 при нарушении. Коммит теперь блокируется, а не только предупреждает.

Детали

  • Скрипты: docstring + лейбл WARN → FAIL, exit-код по флагу нарушений.
  • Манифест .pre-commit-hooks.yaml: убран суффикс «(warn-only)» из имён, описания → «FAIL — коммит запрещён».
  • Fixtures + smoke-кейсы (good/bad) для всех четырёх — раньше эти хуки в smoke вообще не покрывались.
  • README / example-конфиг: задокументированы 4 хука, поправлен счётчик «4 hook» → 8, версии подняты до v0.4.0.

После merge — тег v0.4.0; consumer (baton) бампит rev и убирает verbose: true.

Связано: senokosov/baton#687.

🤖 Generated with Claude Code

## Что `no-underscore-filenames`, `no-private-imports`, `no-private-method-calls`, `no-issue-refs-in-comments` переведены из warn-only в **fail**: `sys.exit(0)` → `exit 1` при нарушении. Коммит теперь блокируется, а не только предупреждает. ## Детали - Скрипты: docstring + лейбл `WARN` → `FAIL`, exit-код по флагу нарушений. - Манифест `.pre-commit-hooks.yaml`: убран суффикс «(warn-only)» из имён, описания → «FAIL — коммит запрещён». - **Fixtures + smoke-кейсы** (good/bad) для всех четырёх — раньше эти хуки в smoke вообще не покрывались. - README / example-конфиг: задокументированы 4 хука, поправлен счётчик «4 hook» → 8, версии подняты до `v0.4.0`. После merge — тег `v0.4.0`; consumer (baton) бампит `rev` и убирает `verbose: true`. Связано: senokosov/baton#687. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
feat: перевести 4 запрета-хука в fail-режим
Some checks failed
ci / smoke (push) Has been cancelled
Details
ci / smoke (pull_request) Has been cancelled
Details
777e382dd6 
no-underscore-filenames, no-private-imports, no-private-method-calls,
no-issue-refs-in-comments: sys.exit(0) → exit 1 при нарушении.
Коммит теперь блокируется, а не только предупреждает.

Добавлены fixtures + smoke-кейсы (good/bad) для всех четырёх — раньше
они в smoke не покрывались. README/example задокументированы, версии
подняты до v0.4.0.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
claude-secretary commented 2026-08-27 11:48:23 +07:00
Author
Owner
Copy link

Закрываю как пустой: содержимое ветки давно в main. merge_base совпадает с вершиной feature/fail-mode-warn-hooks (777e382), diff к main — 0 файлов, 0 строк; fail-режим всех четырёх хуков выпущен ещё в v0.4.0 и описан в README.

Ветка не удалялась и висела открытым PR только по недосмотру.

Закрываю как пустой: содержимое ветки давно в `main`. `merge_base` совпадает с вершиной `feature/fail-mode-warn-hooks` (`777e382`), diff к `main` — 0 файлов, 0 строк; fail-режим всех четырёх хуков выпущен ещё в `v0.4.0` и описан в README. Ветка не удалялась и висела открытым PR только по недосмотру.
claude-secretary closed this pull request 2026-08-27 11:48:28 +07:00
Some checks failed
ci / smoke (push) Has been cancelled
Details
ci / smoke (pull_request) Has been cancelled
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
senokosov/pre-commit-hooks!2
No description provided.