{"id":25732271,"url":"https://github.com/howprogrammingworks/codereview","last_synced_at":"2025-06-11T15:05:54.786Z","repository":{"id":42178817,"uuid":"216641184","full_name":"HowProgrammingWorks/CodeReview","owner":"HowProgrammingWorks","description":"Code Review Examples","archived":false,"fork":false,"pushed_at":"2023-09-24T10:16:17.000Z","size":79,"stargazers_count":7,"open_issues_count":7,"forks_count":7,"subscribers_count":4,"default_branch":"master","last_synced_at":"2023-09-24T18:28:45.968Z","etag":null,"topics":[],"latest_commit_sha":null,"homepage":"","language":"JavaScript","has_issues":true,"has_wiki":null,"has_pages":null,"mirror_url":null,"source_name":null,"license":"mit","status":null,"scm":"git","pull_requests_enabled":true,"icon_url":"https://github.com/HowProgrammingWorks.png","metadata":{"files":{"readme":"README.md","changelog":null,"contributing":null,"funding":".github/FUNDING.yml","license":"LICENSE","code_of_conduct":null,"threat_model":null,"audit":null,"citation":null,"codeowners":null,"security":null,"support":null,"governance":null},"funding":{"patreon":"tshemsedinov"}},"created_at":"2019-10-21T18:46:08.000Z","updated_at":"2022-10-30T09:26:08.000Z","dependencies_parsed_at":"2023-09-24T12:27:01.211Z","dependency_job_id":null,"html_url":"https://github.com/HowProgrammingWorks/CodeReview","commit_stats":null,"previous_names":[],"tags_count":0,"template":null,"template_full_name":null,"repository_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/repositories/HowProgrammingWorks%2FCodeReview","tags_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/repositories/HowProgrammingWorks%2FCodeReview/tags","releases_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/repositories/HowProgrammingWorks%2FCodeReview/releases","manifests_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/repositories/HowProgrammingWorks%2FCodeReview/manifests","owner_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/owners/HowProgrammingWorks","download_url":"https://codeload.github.com/HowProgrammingWorks/CodeReview/tar.gz/refs/heads/master","host":{"name":"GitHub","url":"https://github.com","kind":"github","repositories_count":240788734,"owners_count":19857691,"icon_url":"https://github.com/github.png","version":null,"created_at":"2022-05-30T11:31:42.601Z","updated_at":"2022-07-04T15:15:14.044Z","host_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub","repositories_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/repositories","repository_names_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/repository_names","owners_url":"https://repos.ecosyste.ms/api/v1/hosts/GitHub/owners"}},"keywords":[],"created_at":"2025-02-26T03:40:11.936Z","updated_at":"2025-02-26T03:40:12.609Z","avatar_url":"https://github.com/HowProgrammingWorks.png","language":"JavaScript","funding_links":["https://patreon.com/tshemsedinov"],"categories":[],"sub_categories":[],"readme":"## Code Review\n\n[![Code Review: как это и зачем](https://img.youtube.com/vi/EKL6NiIQ6ZU/0.jpg)](https://www.youtube.com/watch?v=EKL6NiIQ6ZU)\n[![Семинар: Code Review, ответы на вопросы](https://img.youtube.com/vi/AgH4OAKbmkM/0.jpg)](https://www.youtube.com/watch?v=AgH4OAKbmkM)\n\nЗадачи Code Review:\n\n- Улучшение качества кода и поиск лучших решений\n- Повышение квалификации разработчиков\n- Передача знаний и изучение работы коллег для группового владения кодом\n- Поиск лучших решений\n- Граничные случаи\n\nПочему делаем Code Review:\n\n- Замыленный глаз\n- Что, если разработчика переедет автобус?\n- Команда эффективнее разработчиков-одиночек\n\nЧто ищем на Code Review:\n\n- Соответствие структуре и архитектуре проекта\n- Соответствие техническому заданию\n- Читаем ли код коллегами, понятно ли всем именование\n- Как можно повысить информативность кода\n- Возможно ли улучшение метрик сложности алгоритмов\n  - Возможно, стоит изменить структуры данных\n  - Оптимизация алгоритмов\n  - Другие алгоритмы\n- Состояние гонки\n- Утечки памяти\n- Безопасность\n- Антипаттерны\n- Обработка ошибок\n- Лишние зависимости или из замена\n- Улучшение структуры и интеграции программных компонентов\n  - Уменьшение зацепления\n  - Увеличение связности кода\n- Вредные привычки\n\nЧего не ищем (должно быть автоматизировано в CI):\n\n- Стандарты кодирования (кроме первого случая)\n  - Вопросы стиля\n  - Правила именования идентификаторов\n- Повторный код\n- Покрытие тестами\n- Метрики\n\nПроцесс создания и принятия Pull Request:\n\n- Когда подаем?\n- Зачем? У каждого PR должна быть цель\n- Кого добавляем?\n- Как быстро обрабатываем?\n- Как обсуждаем?\n- Консенсус\n- LGTM\n\nПроблемы и о чем нужно помнить:\n\n- Code Review занимает время\n- Раздражает\n- Сведение счетов\n- Доказать кто круче\n- Понятность запроса от инициатора\n  - Просто странный код без всяких комментариев и цели ревью\n  - Почему меня просят про Code Review?\n  - Дополнительные вводные и комментарии\n- Понятность отзыва от ревьюера\n  - Не пишем: это что-то странное тут...\n  - Не сводим к формальностям\n  - Не пишем повестей на много страниц\n- Что бывает, когда много изменений в одном PR\n- Что бывает, когда мало изменений в одном PR\n- Почему лучше сначала планировать разработку\n- Почему лучше делать тесты отдельным коммитом\n- Начните с высокого уровня\n- Пред коммитом сами просмотрите\n- Избегайте новых коммитов во время PR\n- По возможности избегайте альтернативных но не связанных PR\n- Отвечайте на вопросы\n","project_url":"https://awesome.ecosyste.ms/api/v1/projects/github.com%2Fhowprogrammingworks%2Fcodereview","html_url":"https://awesome.ecosyste.ms/projects/github.com%2Fhowprogrammingworks%2Fcodereview","lists_url":"https://awesome.ecosyste.ms/api/v1/projects/github.com%2Fhowprogrammingworks%2Fcodereview/lists"}