core-ds / core-components / 36992082438

83%

push

github

feat/review_skill_team_practice (#2397)

* docs: практика мажорных правок и текст changeset в скилле ревью

Что команда считает мажорным сверх формального списка, как работает с такими
правками, prop из нативных атрибутов (#2307), требования к тексту changeset.
Заголовок ##### в файле на один компонент — не нарушение: extend-changeset.mjs
срезает его и генерирует заново.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* docs: соглашения кода, css и токены в скилле ревью

Новый чек-лист references/code-conventions.md из повторяющихся замечаний
мейнтейнеров и development-faq; подключён к этапу 8.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* docs: тесты, lint:deps и устаревшие документы в скилле ревью

lint:deps корневой package.json не проверяет (depcheck по пакету), правило
чек-листа про root устарело; что команда просит и не просит в тестах;
.github/CONTRIBUTING.md унаследован от ant-design.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* docs: калибровка шума, доступность, дизайн-ревью и утечки в скилле ревью

Чего в ревью этого репозитория не выносят; когда a11y-замечание уместно;
апрув дизайнера на визуальные изменения; утечки и правки общих пакетов.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* docs: база pr, класс находки, проверка исполнением и повторное ревью в скилле

Регресс, новая поверхность или фон; класс, а не экземпляр; воспроизведение,
которое автор может прогнать; команды точечной проверки; этикет повторного
ревью; требования к комментариям, привязанным к строкам.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* docs: правки скилла ревью по комментариям у строк

displayName — только при forwardRef; CSS-переменные — только для темизации.
Убраны дубли про var() в JS и про заголовок ##### в changeset (строка severity
возвращена к исходной), строка про CONTRIBUTING.md и команды запуска: агент
определяет их по окружению. «repro», «на голове PR», «на базе так же»,
«чужой долг... (continued)

1823 of 2280 branches covered (79.96%)

Branch coverage included in aggregate %.

9944 of 11986 relevant lines covered (82.96%)

237.33 hits per line