code-review-101-qwerty1105
🤯 ПОЛНЫЙ README ДЛЯ СТУДЕНТОВ
📋 13 Этапов
- ✅ Создать ветку — любое имя
- ✅ Дождаться файла — assignment.py создаётся автоматически
- ✅ Выполнить задание — заполнить все функции (опять вайбим), получаем результат
- ✅ Добавить PNG файлы — все графики закоммичены
- ✅ Удалить лишние MD — только с отчетом README должно остаться
- ✅ Добавить README — полный отчёт с статистикой по данным, как решали и т.д.
- ✅ Создать PR — правильное описание
- ✅ Добавить рецензентов — двух со-авторов
- ✅ РЕЦЕНЗИРОВАТЬ АДЕКВАТНО — минимум 1 замечание + Request Changes
- ✅ Ответить на замечания — внести улучшения
- ✅ Дождаться Approve — от обоих рецензентов
- ✅ Merge + Удалить ветку — запустить нужную итоговую автопроверку
- ✅ Финальная проверка — все checks зелёные
⚠️ КРИТИЧЕСКИЕ ПРАВИЛА
Студент:
❌ Не засчитывается задание если:
- Нет рецензии (просто нажал merge и не дал советов коллегам)
- README пуст
- Остались лишние файлы
- Отсутствуют PNG
- Код не очень соответствует PEP8
✅ Засчитывается если:
- Все 13 этапов пройдены
- README полный с статистикой
- Двое одобрили ("Approve")
- Ветка удалена
Рецензент (если вы не справились как рецензент то задание в целом вам не засчитывается):
❌ Не засчитывается если:
- Не оставил комментарии
- Сразу нажал "Approve" без правок
- Замечания не конкретны
✅ Засчитывается если:
- Минимум 1 конкретное замечания
- Запросил правки ("Request changes")
- Проверил улучшения
- Потом "Approve"
Преподаватель:
❌ Отклоняет если:
- README формально составлен
- Нет нужных файлов
- Есть лишние markdown файлы
- Рецензия неадекватна
✅ Принимает если:
- Всё оформлено правильно
- Файлы на месте
- Лишних нет
- Рецензия содержательна
📁 Структура README
-
Информация о рецензируемых работах
- gitverse профили
-
Информация о студенте
- ФИО, группа, номер задания
-
Описание задания
- Что нужно анализировать
-
Выполненные работы
- Загрузка данных
- Анализ целевой переменной
- Анализ признаков
- Визуализация
-
Статистика
- Таблица с метриками
- Ключевые числа
-
Ключевые находки
- Выводы из анализа
-
Файлы
- Список созданных и удалённых файлов
🎯 Примеры Рецензии
❌ Плохо:
Посмотри код
Исправь все ошибки
✅ Хорошо:
1. На строке 15: используешь pd.concat()
Рекомендую assign() - он производительнее:
df.assign(target=y)
2. Docstring в load_data() не полный
Добавь возвращаемый тип:
"""Returns: pd.DataFrame"""
3. В visualize_target() нет обработки ошибок
Добавь try-except на случай если целевая переменная не найдена
Как минимум поправь пункты 1 и 2.
Не забудьте ответить рецензенту
✅ Спасибо за замечания!
Внесены улучшения:
1. ✅ Использую assign() вместо concat()
2. ✅ Добавил тип возврата в docstring
3. ✅ Добавил обработку ошибок
Готово к повторной проверке.
📞 Часто Задаваемые Вопросы
Q: Когда появится assignment.py?
A: В течение 30 секунд после создания ветки.
Q: Можно ли merge без рецензии?
A: Технически можно, но задание не засчитается!
Q: Что если рецензент не смог проверить?
A: Замените его на другого студента и попросите новую рецензию
Q: Как узнать что задание зачтено?
A: GitHub Actions покажет ✅ all checks passed
Q: Можно ли исправить рецензию?
A: Да, можно добавить ещё замечания прямо в PR
🏆 Успешное Выполнение
✅ Все 13 этапов выполнены
✅ Рецензия содержательна
✅ README полный
✅ Файлы на месте
✅ Checks прошли
✅ ЗАДАНИЕ ЗАЧТЕНО! 🎉