Правки по итогам code-review deliverable #27: убран backlog-язык из конвенции (Exceptions про edited trigger — теперь постоянное правило), зафиксирована политика Revert/fixup!/squash! для commitlint, README переведён на ссылку в Commit Conventions вместо дублирования правила.
.ai/conventions/engineering-workflow.md — Exceptions: правило ручной проверки PR title без формулировок «может отсутствовать/до подтверждения»; Commit Conventions: шаблонные Revert/fixup!/squash! не освобождаются
README.md — ссылка на Commit Conventions вместо повторного изложения правила ссылки на Issue
Verification
Команда / результат / ограничения: npm run commitlint (config loads, матрица сообщений: revert: x (#123) PASS, Revert "…"/fixup!/без ссылки FAIL, release/merge ignores PASS) — ожидаемое поведение; git diff --check — exit 0. Ограничение: Markdown-only правки, application gates не требуются.
Breaking Changes (если есть)
Нет.
## Summary
Правки по итогам code-review deliverable #27: убран backlog-язык из конвенции (Exceptions про `edited` trigger — теперь постоянное правило), зафиксирована политика `Revert`/`fixup!`/`squash!` для commitlint, README переведён на ссылку в Commit Conventions вместо дублирования правила.
## Linked Issue
Refs #27 (follow-up после ревью merged PR #28).
## Changes
- `.ai/conventions/engineering-workflow.md` — Exceptions: правило ручной проверки PR title без формулировок «может отсутствовать/до подтверждения»; Commit Conventions: шаблонные `Revert`/`fixup!`/`squash!` не освобождаются
- `README.md` — ссылка на Commit Conventions вместо повторного изложения правила ссылки на Issue
## Verification
- Команда / результат / ограничения: `npm run commitlint` (config loads, матрица сообщений: `revert: x (#123)` PASS, `Revert "…"`/`fixup!`/без ссылки FAIL, release/merge ignores PASS) — ожидаемое поведение; `git diff --check` — exit 0. Ограничение: Markdown-only правки, application gates не требуются.
## Breaking Changes (если есть)
Нет.
INFO .ai/conventions/engineering-workflow.md:65-66 — закрыто замечание о шаблонных Revert/fixup!/squash!; формат revert со ссылкой на Issue корректный.
INFO .ai/conventions/engineering-workflow.md:148-151 — убрана оговорка «до подтверждения»: явно указано, что при отсутствии edited trigger reviewer вручную проверяет актуальный PR title.
INFO README.md:166-172 — удалено дублирование правила о ссылке на Issue, добавлена рабочая ссылка на канонический раздел Commit Conventions.
## Итог: Approve
Замечания из issue #27 устранены:
- INFO `.ai/conventions/engineering-workflow.md:65-66` — закрыто замечание о шаблонных `Revert`/`fixup!`/`squash!`; формат `revert` со ссылкой на Issue корректный.
- INFO `.ai/conventions/engineering-workflow.md:148-151` — убрана оговорка «до подтверждения»: явно указано, что при отсутствии `edited` trigger reviewer вручную проверяет актуальный PR title.
- INFO `README.md:166-172` — удалено дублирование правила о ссылке на Issue, добавлена рабочая ссылка на канонический раздел Commit Conventions.
INFO .ai/conventions/engineering-workflow.md:65-66 — закрыто замечание о шаблонных Revert/fixup!/squash!; формат revert со ссылкой на Issue корректный.
INFO .ai/conventions/engineering-workflow.md:148-151 — убрана оговорка «до подтверждения»: явно указано, что при отсутствии edited trigger reviewer вручную проверяет актуальный PR title.
INFO README.md:166-172 — удалено дублирование правила о ссылке на Issue, добавлена рабочая ссылка на канонический раздел Commit Conventions.
## Итог: Approve
Замечания из issue #27 устранены:
- INFO `.ai/conventions/engineering-workflow.md:65-66` — закрыто замечание о шаблонных `Revert`/`fixup!`/`squash!`; формат `revert` со ссылкой на Issue корректный.
- INFO `.ai/conventions/engineering-workflow.md:148-151` — убрана оговорка «до подтверждения»: явно указано, что при отсутствии `edited` trigger reviewer вручную проверяет актуальный PR title.
- INFO `README.md:166-172` — удалено дублирование правила о ссылке на Issue, добавлена рабочая ссылка на канонический раздел Commit Conventions.
ksv741
merged commit d2d1092344 into develop2026-10-12 22:00:21 +03:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
hermes-agent
Summary
Правки по итогам code-review deliverable #27: убран backlog-язык из конвенции (Exceptions про
editedtrigger — теперь постоянное правило), зафиксирована политикаRevert/fixup!/squash!для commitlint, README переведён на ссылку в Commit Conventions вместо дублирования правила.Linked Issue
Refs #27 (follow-up после ревью merged PR #28).
Changes
.ai/conventions/engineering-workflow.md— Exceptions: правило ручной проверки PR title без формулировок «может отсутствовать/до подтверждения»; Commit Conventions: шаблонныеRevert/fixup!/squash!не освобождаютсяREADME.md— ссылка на Commit Conventions вместо повторного изложения правила ссылки на IssueVerification
npm run commitlint(config loads, матрица сообщений:revert: x (#123)PASS,Revert "…"/fixup!/без ссылки FAIL, release/merge ignores PASS) — ожидаемое поведение;git diff --check— exit 0. Ограничение: Markdown-only правки, application gates не требуются.Breaking Changes (если есть)
Нет.
Итог: Approve
Замечания из issue #27 устранены:
.ai/conventions/engineering-workflow.md:65-66— закрыто замечание о шаблонныхRevert/fixup!/squash!; форматrevertсо ссылкой на Issue корректный..ai/conventions/engineering-workflow.md:148-151— убрана оговорка «до подтверждения»: явно указано, что при отсутствииeditedtrigger reviewer вручную проверяет актуальный PR title.README.md:166-172— удалено дублирование правила о ссылке на Issue, добавлена рабочая ссылка на канонический раздел Commit Conventions.Итог: Approve
Замечания из issue #27 устранены:
.ai/conventions/engineering-workflow.md:65-66— закрыто замечание о шаблонныхRevert/fixup!/squash!; форматrevertсо ссылкой на Issue корректный..ai/conventions/engineering-workflow.md:148-151— убрана оговорка «до подтверждения»: явно указано, что при отсутствииeditedtrigger reviewer вручную проверяет актуальный PR title.README.md:166-172— удалено дублирование правила о ссылке на Issue, добавлена рабочая ссылка на канонический раздел Commit Conventions.