Комментарии 10
Так и не увидел какой инструментарий был выбран для code review?
+2
На гитхабе есть встроенный инструмент код-ревью для пулл-реквестов.
-1
Наверное, мы разные вещи называем термином code review
0
К сожалению, такой подход работает только для хорошо мотивированных студентов, к тому же знакомых с гитом.
Нечто подобное я пытался организовать в рамках производственной практики на матфаке Воронежского госуниверситета:
https://github.com/nickkolok/chas-ege/pulls
Что выяснилось:
Нечто подобное я пытался организовать в рамках производственной практики на матфаке Воронежского госуниверситета:
https://github.com/nickkolok/chas-ege/pulls
Что выяснилось:
- В группе оказалась девушка, которая официально изучает немецкий. Смотреть на то, как она все сообщения переводит через Google Translate, было жутковато, требовать знание английского — нереально.
- Подскажите, а есть ли какой-нибудь способ автоматически давать по рукам за концевые пробелы в пулл-реквестах и гибко проверять отступы?
- Можно поручать code review студентам, если задачи не совпадают. И им полезно, и мне нервы-время экономит (а в основном находят ошибки с отступами, типографикой и "белиберду"). Потом, конечно, всё смотрю сам.
0
(2) Насчет пробелов и отступов, для Java могу посоветовать checkstyle и его плагины для популярных сборщиков. Для Javascript (если я правильно понял из репозитория, вам интересен именно он) можно либо прогонять eslint перед тестами, либо воспользоваться готовым сервисом, например, https://houndci.com.
(3) Интересное предложение. Если поручить им еще и тесты писать, а оценки выставлять по степени покрытия, то не жизнь, а сказка будет :)
(3) Интересное предложение. Если поручить им еще и тесты писать, а оценки выставлять по степени покрытия, то не жизнь, а сказка будет :)
+1
(2.1) Да, про eslint знаю, спасибо, вот только там полно легаси-кода, который еслинтовать не надо. Но тема "рефакторинг" у нас тоже есть.
(2.2) Кстати, реально ли заставить Semaphore CI (спасибо за наводку, он реально прост в обращении) проверять каждый входящий пуллреквест? Или это только травис умеет?
(3) Веду у первого курса плюсы (меня поставили со второго семестра), сочиняю для них лабы. Даже программу слегка поменял, ускоренно пройдя функции, чтоб заставить их писать проостенькие юнит-тесты. Написал код, который в строке инвертирует регистр согласных — вынеси в функцию и покрой тестами!
(2.2) Кстати, реально ли заставить Semaphore CI (спасибо за наводку, он реально прост в обращении) проверять каждый входящий пуллреквест? Или это только травис умеет?
(3) Веду у первого курса плюсы (меня поставили со второго семестра), сочиняю для них лабы. Даже программу слегка поменял, ускоренно пройдя функции, чтоб заставить их писать проостенькие юнит-тесты. Написал код, который в строке инвертирует регистр согласных — вынеси в функцию и покрой тестами!
0
git config core.whitespace (см. справку) — скорее всего то, что вы хотите по поводу пробелов на концах строк :)
0
Зарегистрируйтесь на Хабре, чтобы оставить комментарий
Как мы использовали Git, CI и code review в учебном процессе