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

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

Как у Вас реализована проверка качаства кода на соответствеие корпоративным стандартам (отступы, табы, пробелы, определенные части кода)

На каком этапе происходит эта проверка ? Какие инструменты инспользуете (плюсы, минусы). Что бы хотелось реализрвать ?

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

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

Pre-commit hook with checkstyle

Gres
checkstyle

это что? Тула какая?

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

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

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

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

Dmitry Shnyrev
это что? Тула какая?

link

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

Gres
Pre-commit hook with checkstyle

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

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

Да

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

Да

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

wilder
Как у Вас реализована проверка качаства кода на соответствеие корпоративным стандартам (отступы, табы, пробелы, определенные части кода)

anupj/Apex-Code-Convention

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

Andrii Muzychuk
wilder
Как у Вас реализована проверка качаства кода на соответствеие корпоративным стандартам (отступы, табы, пробелы, определенные части кода)

anupj/Apex-Code-Convention

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

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

Interesting information? Help us, post link to social media..