Качество кода и корпоративные стандарты

Я был на многих проектах и только а одном было хоть что-то сделано. 
Сейчас на моем последнем проекте мы стараемся ввести туже самую систему (только толкового стандарта пока у нас нет).
В общем как делали. Работаем через git и каждая таска в отдельном бранче. Разработчик заканчивает таску и делает pull request в master ветку (к примеру). В этом PR он назначает 1 или больше Reviewers которые должны проверить код по diff и заапрувить PR. Если проверяющих устраивает и все апрувят изменения, то ветка уходит в основную ветку. Если нет, тут же в PR (мы работаем с Bitbucket и там отлично интерфейс сделан) добавляем комменты что поправить. Разработчик уходит править и обновляет PR. Процесс начинается с начала. 

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

Pre-commit hook with checkstyle

[quote="Gres"]checkstyle[/quote]
это что? Тула какая?

[quote="Dmitry Shnyrev"]Я был на многих проектах и только а одном было хоть что-то сделано. 
Сейчас на моем последнем проекте мы стараемся ввести туже самую систему (только толкового стандарта пока у нас нет).
В общем как делали. Работаем через git и каждая таска в отдельном бранче. Разработчик заканчивает таску и делает pull request в master ветку (к примеру). В этом PR он назначает 1 или больше Reviewers которые должны проверить код по diff и заапрувить PR. Если проверяющих устраивает и все апрувят изменения, то ветка уходит в основную ветку. Если нет, тут же в PR (мы работаем с Bitbucket и там отлично интерфейс сделан) добавляем комменты что поправить. Разработчик уходит править и обновляет PR. Процесс начинается с начала. 

На счет корпоративных стандартов - я, так сложилось, работаю или напрямую с заказчиками или в составе небольших стартапов. Ни те ни другие особо не следят за стандартами - правила вырабатываются на лету и просто все про них знают. Проверяем в основном адекватность кода, нейминг, отступы. Тулы для этого даже ни разу в глаза не видел.[/quote]

Ну про стандарты работы это совсем другая тема. Я работаю именно так.

Про корпоративные стандарты печалька :( Пора внедрять.

[quote="Dmitry Shnyrev"]это что? Тула какая?[/quote]
[url=http://guglite.ru/?q=checkstyle]link[/url]

От Gres, подебал :D
Я что-то не подумал что "checkstyle" это имя собственное какой тулы. 
Блин, они оригинальнее название не могли придумать? :D 

[quote="Gres"]Pre-commit hook with checkstyle[/quote]

Есть опыт личного использования ?

[quote="wilder"]Есть опыт личного использования ?[/quote]
Да

[quote="Gres"][quote="wilder"]Есть опыт личного использования ?[/quote]
Да[/quote]

Чудно. Поделиться не хочешь ?

[quote="wilder"]Как у Вас реализована проверка качаства кода на соответствеие корпоративным стандартам (отступы, табы, пробелы, определенные части кода)[/quote]
[url=https://github.com/anupj/Apex-Code-Convention/blob/master/ApexCoding.txt]anupj/Apex-Code-Convention [/url]

[quote="wilder"]На каком этапе происходит эта проверка ? Какие инструменты инспользуете (плюсы, минусы). Что бы хотелось реализрвать ?[/quote]Code review

[quote="Andrii Muzychuk"][quote="wilder"]Как у Вас реализована проверка качаства кода на соответствеие корпоративным стандартам (отступы, табы, пробелы, определенные части кода)[/quote]
[url=https://github.com/anupj/Apex-Code-Convention/blob/master/ApexCoding.txt]anupj/Apex-Code-Convention [/url]

[quote="wilder"]На каком этапе происходит эта проверка ? Какие инструменты инспользуете (плюсы, минусы). Что бы хотелось реализрвать ?[/quote]Code review[/quote]

Это все как-бы работает только в ручном режиме. Но уже что-то. Пасиб