LINUX.ORG.RU

История изменений

Исправление Aber, (текущая версия) :

Какой язык? Есть статические анализаторы? Линтеры? Конечно внедрение такого требует времени, но оно того стоит, если разумеется, соблюдать основное правило – не заливать новые ворнинги и подключить к CI чтоб падало при добавлении нового ворнинга. И постоянно форсить добавления новых правил все более и более ограничивающих свободы написания кода, потому как способов написать дичь люди знают много, но где-то 90% обычно покрывается статическим анализаторами.

Я сам работал в команде где был код ревью, так вот правила там плавающие были, в зависимости от настроения людей, когда делать нечего а на трекере задачи какие-то нужно отметить то все кидались ревьювить код, это был ад чтоб что-то закомитить :) А когда работы много можно было дичь закомитить и никто даже не колыхнется при реврью – одобрено.

Исходная версия Aber, :

Какой язык? Есть статические анализаторы? Линтеры? Конечно внедрение такого требует времени, но оно того стоит, если разумеется, соблюдать правило – не заливать новые ворнинги и подключить это к CI чтоб падало при добавлении нового ворнинга. И постоянно форсить добавления новых правил все более и более ограничивающих свободы написания кода, потому как способов написать дичь люди знают много, но где-то 90% обычно покрывается статическим анализаторами.

Я сам работал в команде где был код ревью, так вот правила там плавающие были, в зависимости от настроения людей, когда делать нечего а на трекере задачи какие-то нужно отметить то все кидались ревьювить код, это был ад чтоб что-то закомитить :) А когда работы много можно было дичь закомитить и никто даже не колыхнется при реврью – одобрено.