forumСоздать тему

Принимаем переданный код — есть чек-лист код-ревью для не-разработчиков?

EErcan T***УчастникУчастник сообщества
Дата регистрации
апр. 2022 г.
Сообщение
138
#1

мы компания из 14 человек в Калифорнии делаем софт для логистики. разошлись с прежним агентством; они сдали последний этап проекта кастомного клиентского портала который длился восемь месяцев и обошелся нам в общей сложности в 65.000 доллларов и репозиторий кода перенесли на свой аккаунт, но в штате у нас нет ни одного человека с должностью разработчика, да и у меня самого управленческий бэкграунд.

договорились с новой командой фрилансеров, но прежде чем выплатить старому агентству финальные 10.000 долларов хочу лично понять в каком состоянии то что мы приняли. понятно, что влезть в код и построчно выискивать логические ошибки я не смогу но есть ли осязаемый чек-лист проверки исходного кода, по которому технически неподкованный менеджер может визуально оценить порядок в файлах документацию или общую дисциплину сдачи? о чем спрашивать при приемке и что искать поиском по репозиторию?

LLeyla Y***УчастникУчастник сообщества
Дата регистрации
март 2026 г.
Сообщение
61
Самый полезный#2

Короткий ответ: оценить качество кода построчно вы в одиночку не сможете, но убедиться, что репозиторий передан полностью, может работать автономно и соответствует базовым стандартам, вполне реально и без технических знаний. До финальной выплаты критически важно подтвердить, что проект поднимается без привязки к третьим лицам.

Начните чек-лист со следующих шагов: 1) Наличие инструкции по установке. В репозитории должен быть 'README' или дока по развертыванию. Передайте этот документ новому разработчику со стороны, который вообще не знает проект, и попросите поднять систему на чистом компьютере строго по этим шагам. Если без звонка в агентство запустить не удается — сдача не завершена. 2) Проверка лицензий и авторских прав. В коде не должно быть проприетарных кусков, скопированных у других клиентов агентства, просроченных коммерческих библиотек или запрещенных опенсорс-лицензий.

3) Поиск зашитых паролей и ключей. Вбейте в поиск по коду слова 'password', 'secret', 'api_key'. Убедитесь, что реальные пароли от БД или ключи от боевого эквайринга не вбиты прямо текстом в код, а подтягиваются через переменные окружения. 4) История изменений и владение. Посмотрите, зафиксированы ли коммиты с внятными описаниями; если сотни файлов залиты в один день с комментариями вроде 'финал' или 'правка', значит, агентство скрывает реальный процесс разработки. Не подписывайте выплату, пока эти четыре пункта не проверит новая команда с фиксацией в акте приемки.

SSelin K***Участник
Должность
HR-специалист
Отрасль
Э-коммерция
Тип организации
производство
Дата регистрации
сент. 2024 г.
Сообщение
42
#3

Самое критичное — это автономность. Работает ли код на вашем сервере без привязки к внешним серверам агентства, их аккаунтам разработчиков или личным почтам сотрудников? Если внешний сервис уведомлений или ключ от сервера висят на их аккаунте, потом система просто встанет.

KKader A***Участник
Должность
Директор магазина
Отрасль
Косметика
Тип организации
компания с двумя филиалами
Дата регистрации
нояб. 2022 г.
Сообщение
183
#4

Даже без технических знаний можно запустить автоматические сканеры кода. Включите бесплатное сканирование безопасности прямо в вашем репозитории. Эти инструменты выдадут отчет с открытыми уязвимостями и устаревшими пакетами; его можно прямо переслать агентству со словами: «Вычищайте».

ÖÖzgür B***УчастникУчастник сообщества
Дата регистрации
февр. 2023 г.
Сообщение
34
#5

В прошлом году на аналогичной передаче проекта за 40.000 долларов мы наступили на те же грабли. Думали, что забрали код, а там автотестов не было вообще. Новая команда попросила сверху 12.000 долларов и 6 недель, просто чтобы разобраться и покрыть всё тестами. Обязательно проверьте, есть ли папка с тестами и не пустая ли она.

ZZafer S***Участник
Должность
Продавец-консультант
Отрасль
Страхование
Тип организации
индивидуальный предприниматель
Дата регистрации
дек. 2025 г.
Сообщение
67
#6

Беглый просмотр репозитория менеджером дает лишь ложное спокойствие. Утечку памяти или дыру в запросах к базе по названиям файлов вы не увидите. Намного надежнее заплатить новой команде 500–1.000 долларов за письменный отчет о техническом приеме проекта.

AAslıhanУчастник
Должность
Производитель одежды
Дата регистрации
апр. 2024 г.
Сообщение
102
#7

Прямо завтра созвонитесь с новым фрилансером на час с демонстрацией экрана. Пусть он с нуля развернет и запустит систему, накатит бэкап базы данных и выполнит любое действие в админке. Если справится — можно спокойно отдавать финальный расчет.

CCaner B***УчастникУчастник сообщества
Дата регистрации
дек. 2024 г.
Сообщение
1
#8

Когда старое агентство сдавало проект, всё выглядело идеально. Спустя два месяца сменили маршрутизацию домена и выяснили, что интеграция пушей висела на личной кредитке владельца агентства. Карту заблокировали — уведомления упали. Убедитесь, что все аккаунты внешних сервисов переведены на корпоративные почты вашей компании.

BBeren V***Участник
Должность
Специалист технического сервиса
Отрасль
Реклама и продвижение
Тип организации
Компания на 120 человек
Дата регистрации
март 2024 г.
Сообщение
123
#9

даты последних коммитов в репе тоже гляньте. честно если за два дня до конца договора залили тыщу файлов одним махом они и сами могли забыть че там внутри и закрывали все в дикой спешке.

P.S.: вопрос уже задавали ниже, ответ во втором сообщении.

MMurat B***Новый участник
Должность
Специалист службы поддержки клиентов
Отрасль
Морепродукты
Тип организации
кооператив
Дата регистрации
июнь 2026 г.
Сообщение
123
#10

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

AAhmet N***Эксперт
Должность
Продавец-консультант
Отрасль
Строительство
Тип организации
компания в составе холдинга
Дата регистрации
июль 2022 г.
Сообщение
153
#11

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

Внедрение процесса управления изменениями не замедляет работу, а ускоряет её. Надеюсь, это будет полезно.

HHüseyin Ş***Участник
Должность
Сетевой администратор
Отрасль
Упаковка
Тип организации
компания в составе холдинга
Дата регистрации
июль 2023 г.
Сообщение
81

Doki · Фирменный стиль · 2025

#12

не упустите: Все кто торопится с чек-лист проверки исходного кода сотыкаются на одном и том же.

это мое мнение, не утверждаю что оно единственно верное.

UUğur D***Участник
Должность
Оператор ввода данных
Отрасль
Медицинские услуги
Тип организации
индивидуальный предприниматель
Дата регистрации
янв. 2022 г.
Сообщение
2
#13

Прочитал этот ответ и выдохнул: значит, проблема не только у меня. Первые три месяца все идет хорошо, проблемы всплывают на четвертом.

Конечно, если ваша ситуация отличается, всё меняется.

ZZehra Y***Участник
Должность
Директор по маркетингу
Отрасль
Производство мебели
Тип организации
компания из 20 человек
Дата регистрации
май 2024 г.
Сообщение
324
#14

Ответ выше точно передал суть вопроса. Если скоуп растет должен расти либо срок либо бюджет. Третьего не дано.

Интересно, есть ли те, кто делает иначе.

KKader A***Новый участник
Должность
Сетевой администратор
Отрасль
Консалтинг
Тип организации
среднее предприятие
Дата регистрации
сент. 2026 г.
Сообщение
10
#15

В теории верно, но на практике так не работает. Первые три месяца все идет хорошо, проблемы всплывают на четвертом.

Исправьте, если я ошибаюсь.

NNazlı G***Участник
Должность
Бухгалтер
Отрасль
Сельское хозяйство
Тип организации
среднее предприятие
Дата регистрации
нояб. 2023 г.
Сообщение
176
#16

Попробую.

CCem I***УчастникУчастник сообщества
Дата регистрации
сент. 2025 г.
Сообщение
4
#17

Обсуждение размылось, давайте структурируем. Если критерии приемки не прописаны, момент завершения работы становится предметом спора.

UUfuk S***Участник
Должность
Первичная бухгалтерия
Отрасль
Сельское хозяйство
Тип организации
кооператив
Дата регистрации
июнь 2022 г.
Сообщение
74

Doki · Поддержка реагирования на инциденты · 2025

#18

Пишу это, чтобы вы не совершили ту же ошибку. Доступы должны открываться на компанию, а не на личные аккаунты.

Прежде чем принимать решение, посмотрите, какие данные у вас есть.

DDamla M***Участник
Должность
Полевой торговый представитель
Отрасль
Недвижимость
Тип организации
среднее предприятие
Дата регистрации
июль 2025 г.
Сообщение
63

Doki · Мобильное приложение · 2023

#19

Извините, но это верно не во всех случаях. кстати самое дешевое предложение обычно самое непродуманное.

Исправьте если я ошибаюсь.

ÖÖzge A***ЭкспертУчастник сообщества
Дата регистрации
апр. 2025 г.
Сообщение
42
#20

Хочу предупредить. Прежде чем принимать решение, посмотрите, какие данные у вас есть.

Я бы пошел этим путем.

Написать ответ