mirror of
https://github.com/Matysh/houseplan-card
synced 2026-09-30 19:58:50 +00:00
docs(process): name today's monolith tolerance in the rebase rule (#698 r1)
CODE-REVIEW-698-r1 Medium: the canon said the merged monolith numbers are judged «by the band test (#699)», but #699 is not merged — today compareWithBaseline judges five numbers exactly and only dist/ bytes with a band. The paragraph now says so: dev's side of the baseline turns the candidate's Validate red when the task itself changed those numbers — the same return to the author as before, after Validate instead of before the review; the band for all six numbers is #699. The comment on UPSTREAM_WINS says the same. Issue: #698 User-Visible: no Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018qZfe7YS4rqEMKoVeS3GKd
This commit is contained in:
+7
-3
@@ -392,9 +392,13 @@ dev, ни при публикации документа код-ревью: ин
|
||||
вместе с любым другим путём — прежний отказ с перечнем файлов. Ещё два общих
|
||||
файла не конфликтуют по смыслу (#698): записи ченджлогов в `## Unreleased`
|
||||
объединяет встроенный драйвер `merge=union` (`.gitattributes`), а на конфликте в
|
||||
`scripts/monolith-baseline.json` ребейз берёт сторону `dev` — число для
|
||||
объединённого дерева судит тест полосы на Validate (#699). Ни то, ни другое не
|
||||
входит в patch-id кандидата слияния: вердикт к работе задачи остаётся в силе. Шаг Validate
|
||||
`scripts/monolith-baseline.json` ребейз берёт сторону `dev` — числа
|
||||
объединённого дерева судит гейт связности монолита (`npm run lint:unused`) на
|
||||
Validate кандидата. Сегодня пять чисел исходника он судит точно, байты `dist/` —
|
||||
с полосой 2 000 Б; поэтому сторона `dev` в базе краснеет, если задача сама
|
||||
меняла эти числа, — это прежний возврат автору, только после Validate, а не до
|
||||
ревью. Полоса у всех шести чисел — #699. Ни ченджлоги, ни база не входят в
|
||||
patch-id кандидата слияния: вердикт к работе задачи остаётся в силе. Шаг Validate
|
||||
«индекс ревью совпадает с каталогом» красит push в `dev`, где `INDEX.md`
|
||||
расходится с каталогом (на issue-ветках не судится: их переписывает конвейер).
|
||||
Правка `docs/reviews/`
|
||||
|
||||
@@ -35,8 +35,9 @@ export const REVIEWS_INDEX_PATH = 'docs/reviews/INDEX.md';
|
||||
/**
|
||||
* #698: данные, в которых на конфликте права сторона `dev`. База метрик
|
||||
* монолита — снимок чисел, а не решение задачи: правильное значение для
|
||||
* объединённого дерева не «наше» и не «их», и судит его тест полосы на
|
||||
* Validate кандидата (#699), а не автор руками. Ченджлоги сюда не входят —
|
||||
* объединённого дерева не «наше» и не «их», и судит его гейт связности на
|
||||
* Validate кандидата (`lint:unused`; полоса у всех чисел — #699), а не автор
|
||||
* руками. Ченджлоги сюда не входят —
|
||||
* их объединяет `merge=union` в `.gitattributes`, и конфликта не бывает.
|
||||
*/
|
||||
export const UPSTREAM_WINS = Object.freeze(['scripts/monolith-baseline.json']);
|
||||
|
||||
Reference in New Issue
Block a user