Tasks
Tasks copied to clipboard
BlizD
Удалить комментарии разработчиков
В продолжении обсуждения комментариев в коде, а также задачи #155. Предлагаю вариант "и вашим, и нашим":
- удалить те именные комментарии из кода, что старше трех месяцев;
- проводить подобную чистку с периодичностью 2-3 месяца.
Вспоминая обновления типовых. Как же у меня подгорало когда в большом количестве модулей изменяли пробелы или добавляли их. Конечно в тех модулях где наших изменений не было то все ок. А там где были и модуль большой это был вырви глаз. Но это лирическое отступление. От предложения не отказываюсь, но предлагаю это делать в периоды уменьшения кол-во доработок новых.
У меня тяжелые воспоминания с точками в конце комментариев. :-) Однако оглядываясь назад подобные изменения в типовых больше не пугают: благо известны средства трехстороннего сравнения модулей, благодаря которым проблема точек и пробелов сводится на нет.
Что касается предложения, то на мой взгляд эти изменения как раз таки нужны в период большого количества доработок, когда много приходится работать с кодом. Оглядываясь на лирическое отступление можно попросить разработчиков перед отправкой cf разрешить конфликты с веткой dev.
Идеально, это было бы так.
В ветке master перед выпуском релиза запускаешь какой то скрипт, который вырезает из кода блоки закомментированного текста для тех объектов у которых нет замков.
Пока к комментариями у меня позиция с ними лучше, чем без них=) Но думаю такую чистку надо проводить согласен.