feat: Add PR template (Phase 8.2) #69

Merged

claude merged 1 commits from feature/pipeline-templates into main 2026-02-14 19:08:18 +01:00

Conversation 1 Commits 1 Files Changed 1 +12

claude commented 2026-02-14 19:06:15 +01:00

Owner

Copy Link

Что изменилось

PR template для стандартизации описаний PR.

Зачем

Phase 8.2 — Standardized Pipeline Templates.

## Что изменилось PR template для стандартизации описаний PR. ## Зачем Phase 8.2 — Standardized Pipeline Templates.

claude added 1 commit 2026-02-14 19:06:15 +01:00

feat: add PR template (Phase 8.2) ... 664a4f1f12

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

claude commented 2026-02-14 19:06:27 +01:00

Author

Owner

Copy Link

AI Code Review

Обзор PR

Статус: ✅ Одобрено — качество хорошее

Позитивное

• Шаблон охватывает критические области (blast radius, секреты, форматирование)
• Чеклист связан с вашей toolchain (tofu, Conftest, SOPS)
• На русском — соответствует локализации проекта

Замечания (минор)

1. Blast radius — уточнить примеры

## Blast radius
<!-- Примеры:
- VM 201-204 (VM-as-a-Service slots)
- K8s namespaces (dev/staging/prod)
- Network rules (DNAT на bare_srv_1)
Есть ли деструктивные изменения? -->

Авторы PR смогут быстрее оценить влияние.

2. Добавить проверку terraform plan

- [ ] `tofu plan` проверен (нет деструктивных ресурсов)

Критично, чтобы автор явно подтвердил, что -/+ destroy нежелательны.

3. State migration (опционально)

- [ ] State файлы обновлены (если manual state edit)

Если ваш workflow предполагает state moves для рефакторинга.

Рекомендация

Добавьте ⚠️ блок для деструктивных операций:

⚠️ **ВНИМАНИЕ:** Если PR содержит `aws_instance destroy` или подобное, 
потребуется ручное утверждение + запланированное окно downtime.

---

Действие: Merge готов. Рекомендую затем обновить ROADMAP.md (фаза 8.2 — статус DONE).

---

Automated review by Claude Haiku | advisory, does not block merge

## AI Code Review ## Обзор PR **Статус:** ✅ Одобрено — качество хорошее ### Позитивное - Шаблон охватывает критические области (blast radius, секреты, форматирование) - Чеклист связан с вашей toolchain (tofu, Conftest, SOPS) - На русском — соответствует локализации проекта ### Замечания (минор) **1. Blast radius — уточнить примеры** ```markdown ## Blast radius <!-- Примеры: - VM 201-204 (VM-as-a-Service slots) - K8s namespaces (dev/staging/prod) - Network rules (DNAT на bare_srv_1) Есть ли деструктивные изменения? --> ``` Авторы PR смогут быстрее оценить влияние. **2. Добавить проверку terraform plan** ```markdown - [ ] `tofu plan` проверен (нет деструктивных ресурсов) ``` Критично, чтобы автор явно подтвердил, что `-/+ destroy` нежелательны. **3. State migration (опционально)** ```markdown - [ ] State файлы обновлены (если manual state edit) ``` Если ваш workflow предполагает state moves для рефакторинга. ### Рекомендация Добавьте `⚠️` блок для деструктивных операций: ```markdown ⚠️ **ВНИМАНИЕ:** Если PR содержит `aws_instance destroy` или подобное, потребуется ручное утверждение + запланированное окно downtime. ``` --- **Действие:** Merge готов. Рекомендую затем обновить ROADMAP.md (фаза 8.2 — статус DONE). --- _Automated review by Claude Haiku | advisory, does not block merge_

claude merged commit 180996d337 into main 2026-02-14 19:08:18 +01:00

claude deleted branch feature/pipeline-templates 2026-02-14 19:08:18 +01:00

claude referenced this issue from a commit 2026-02-14 19:08:18 +01:00

Merge pull request 'feat: Add PR template (Phase 8.2)' (#69) from feature/pipeline-templates into main

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

admin claude

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: claude/infrastructure#69

No description provided.