All streams
Search
Write a publication
Pull to refresh
17
0
Eugene Ilyin @IlyinEugene

Team Leader, FrontEnd Developer

Send message
так идея не в том чтобы отловить ошибки а чтобы их не было. Каждый за собой следит локально, а для нарушителей есть проверка удаленно.

>сложнее поддерживать, если пишете свои правила для проверки

Да в этом есть сложность. У нас в офисе на каждой машине есть папка смонтированная с сервера, для самых разных нужд. В ней же и лежат правила проверки. Таким образом все смотрят в одно место.

Вроде есть какая то возможность настроить PEAR чтобы он искал правила удаленно — это может понадобиться для распределенной команды. Но я пока с этим не разбирался.
У меня две проверки, при коммите на локальном и при добавлении на удаленный. На удаленном отправляется email старшему разработчику.
не совсем понял? в локальном репозитории или удаленном? и почему именно в мастере?
>Нельзя человека, который пишет натюрморты называть плохим художником, только по тому, что вы предпочитаете пейзажи.

Правильно, нельзя. Но я напрямую никого не обвиняю в своем посте. Просто это наболело и те кому приходилось столкнуться с этим наверняка поймут меня.
> Соблюдать конвенцию — не слишком сложно, и стоит просто приучится делать это на автомате.

Мы уже используем это средство в нашей компании. И то что я регулярно вижу ошибки и тут же исправляю их привело к тому что я стал очень редко допускать их ) Остальные сотрудники отмечают примерно тоже самое.
> Во-первых, если человек использует способ оформления кода отличный от вашего, то не стоит его называть говнокодером.

А какой стиль правильный? Вы никогда никому не докажете что ваш стиль лучше чьего-то. Для этого и есть стандарты и их следует соблюдать.

Information

Rating
Does not participate
Location
Россия
Registered
Activity