Skip to content

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

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 есть в репозитории
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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