Git points
- Название ПР корректно, кратко и достаточно
- Есть ли комментарий к изменениям, не относящимся к основной задаче реквеста напрямую?
- Правильно ли выбрана ветка для вливания реквеста
- Соблюдается ли правило – одна ветвь – один коммит?
General points
-
Работает ли код
a. Код выполняет ту и ТОЛЬКО ту задачу, которая задана
b. Код не эффектит остальные части приложения незапланированным образом
c. Легко понять, что делает код (KISS, YANGNI) благодаря его структуре и названиям
-
Использованы имеющиеся на проекте библиотеки, а не изобретено кастомное решение
-
Нет повтора кода (DRY)
-
Названия переменных, файлов и тд с использованием naming conventions проекта
-
Соблюдается code conventions принятые на проекте
-
Есть локализация (опционно)
-
Код очищен от неиспользуемых переменных и комментариев (комментарий НЕ должен описывать КАК это работает, А объяснять ЗАЧЕМ)
-
Нет hardcode значений
-
Константы и модели данных вынесены в отдельные файлы
Angular points
-
Типизация выполнена
-
Использованы стандарты и гайдлайны фреймворка
-
Использованы best practice фреймворка
a. Желательно использовать модульную архитектуру
b. Использовать trackBy с *ngFor
c. Использовать ChangeDetectionStrategy.OnPush при возможности
d. Отписываться от потоков или (что лучше) использовать пайп async
e. Использовать CDK(Component Development Kit) Virtual Scroll для больших списков
f. Стараться оставлять подписку пустой и не делать подписку в подписке! .subscribe(()=>obs$.subscribe())
g. Использовать интерфейсы
h. Избегать мутирования объектов
i. Стартовые данные описывать не в конструкторе, а в хуке ngOnInit
j. При необходимости использовать Route Guards для навигации
k. Использовать *ngIf для скрытия эл-тов, а не CSS
l. Предпочитать реактивные формы шаблонным
Style points
- Использован БЭМ
- Селекторы логически сгруппированы
- Использованы заданные проектом константы
- Избегать использования !important, absolute positioning
- Избегать задания фиксированной высоты
- Избегать использования тэгов в качестве селекторов вместо классов
- Переписывание стилей библиотек в специальных override файлах – не использовать ::ng-deep