Skip to content

Latest commit

 

History

History
23 lines (16 loc) · 2.2 KB

FOR_REVIEWERS.md

File metadata and controls

23 lines (16 loc) · 2.2 KB

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

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.
  • Проверяйте наличие вызова родителя с возвратом результата, т.е . = ..(), либо наличие "хинтов" в виде возвращаемого - тогда прок "точка" скорее всего будет не нужен.

Пробелы

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