На что смотреть при ревью

Warning

Раздел устарел, почти ничего не охвачено, надо удалить или актуализировать.

Здесь будет коротко и с расчетом что у вас уже есть опыт, описано на что важно обратить внимание проводя ревью какого-либо ПР.

<<

Если видите нечто H << "hello" , то с большой вероятностью что там должен быть использован to_chat().

New

/some_atom/New() - Мы перешли на использование atom_init и его "HINT", atom_init_late для класса /atom. Есть редкие случаи когда таки нужен New, но это речь про вещи которым кровь-износу важно быть к сразу поднятию сервера и до инициализации контроллеров (например что-нибудь связанное с клиентами).

new

var/obj/O = new(myarg = 3) здесь будет ошибка с myarg = 3. Поддержка фичи "named arguments" потрачена.

atom_init и его аргументы.

Если есть аргументы, то первым ВСЕГДА обязан быть mapload - atom_init(mapload, ...).
Важно чтобы аргументы были в одинаковой последовательности и количестве у всего класса и его подклассов когда они используются внутри того или другого прока с пересылкой через new.
Проверяйте наличие вызова родителя с возвратом результата, т.е . = ..(), либо наличие "хинтов" в виде возвращаемого - тогда прок "точка" скорее всего будет не нужен.

Пробелы

Поглядывайте чтобы их не использовали заместо табов для уровня блоков кода.