Skip to content

Merge discipline: up-to-date branches and a PR checklist (main went red after #67) #105

Closed

Description

@axisrow

Part of #87.

⚠️ Выводы не проверены человеком. Ишью заведено по аудиту, который сделал Claude Code. Проверка ниже — одна ссылка или одна команда.

Что случилось

За всю историю форка это единственный красный main, но и защиты от повторения нет.

Сопутствующее, по аудиту 10 последних PR:

Как проверить (один раз)

Открыть run #112 по ссылке выше. Или локально:

git checkout 70e3053 && pnpm vitest run test/main/utils/jsonl.test.ts; git checkout -

Сейчас: 2 failed (skips teammate relays when picking firstUserMessage, picks a later real user message when the session opens with a relay).

Что сделать

  1. Settings → Branches → правило для main: «Require status checks to pass» (проверка ci) и «Require branches to be up to date before merging» — или merge queue. В CI: typecheck / lint / test on push + PR #6 branch protection названа необязательной для репо с одним мейнтейнером; этот случай показывает её цену, решение стоит принять явно.
  2. .github/pull_request_template.md с коротким чек-листом:
    • ветка обновлена с main;
    • описание совпадает с итоговым кодом;
    • UI-изменение — есть тест на уровне элементов отображения;
    • для оставленных компромиссов заведены follow-up issue.

Acceptance

  • Правило для main включено (или явно отклонено с причиной в этом ишью)
  • Шаблон PR есть в репозитории

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions