Size: a a a

2018 August 12

l

lokinay in QA Сибирь
Ну,лично у меня бич с разрабами в том,что не хотят писать доку,неважно какую(дока к сервису,дока к тикету,apidoc и пр.).Получается в итоге тестируй то,не знаю что.Не хотят писать unit-тесты.Не хотят проводить ревью своего кода.Самое обидное,что понимают,что я правильно требую от них - по итогу тупо ленивые жопы :)
источник

ОН

Олег Неумывакин in QA Сибирь
Нет, ну code review то нужно делать, неужели один человек может без review влить все что угодно в мастер?
источник

E

Ekaterina in QA Сибирь
Олег Неумывакин
@kami_nary lokinay Поделитесь своим опытом? В чем вы хотите убедить разработку?
Мне особо не в чем разработку убеждать. Разве что хочется, чтобы юнит-тесты на самом деле писались, а не только все соглашались, что они нужны)
источник

l

lokinay in QA Сибирь
Вот так и происходит.на одного из них я постоянно бомблю,что его к исходникам нельзя пускать)
источник

l

lokinay in QA Сибирь
Моя проблема с разрабами не столько в убеждении.Убедить в необходимости то убедил,осталось побороть их лень)
источник

AS

Ann Shumeyko in QA Сибирь
lokinay
Ну,лично у меня бич с разрабами в том,что не хотят писать доку,неважно какую(дока к сервису,дока к тикету,apidoc и пр.).Получается в итоге тестируй то,не знаю что.Не хотят писать unit-тесты.Не хотят проводить ревью своего кода.Самое обидное,что понимают,что я правильно требую от них - по итогу тупо ленивые жопы :)
Мне кажется, всем тестировщикам это очень знакомо:) у меня на проекте тоже поначалу так было. Но сейчас, такое очень редко случается, чтоб разработчик не комментил зарезолвенную задачу или не актуализировал страничку с функционалом на конфлюенсе.
источник

ОН

Олег Неумывакин in QA Сибирь
Погоди, но ведь это настраватеся в битбакете например. Ты можешь пойти с тим лидом к админам и настроить это?
источник

ОН

Олег Неумывакин in QA Сибирь
Минимум два ревьювера и уже станет лучше.
источник

E

Ekaterina in QA Сибирь
@oneumyvakin, а расскажи на сколько обязательно должно быть ревью по твоему мнению?
источник

ОН

Олег Неумывакин in QA Сибирь
В терминологии RFC - "MUST", т.е. обязательно и без вариантов оно должно быть всегда и везде. Сколько людей должны сказать ок - это дискусиионный вопрос, минимум понятно один, а вот два и более зависит от проекта.
источник

ОН

Олег Неумывакин in QA Сибирь
К сожалению, я сам никогда битбакет или gitlab не настраивал, поэтому незнаю насколько гибко можно установить правила
источник

E

Ekaterina in QA Сибирь
У нас пока дискуссии чисто теоретические. Есть опасение, что ревью мелких задач/коммитов испортит у всех впечателнение от ревью
источник

ОН

Олег Неумывакин in QA Сибирь
По идее, может быть какой-то компонент который лежит в общем репозитории но он чертовски сложен и одного человека может быть недостаточно для ревью, причем может быть не достаточно одого любого человека, а должен посмотреть нампример тим лид, или это должен быть обязательно человек из какой-то группы.
источник

E

Ekaterina in QA Сибирь
Ну да, получается, что в зависимости от задачи возникает много нюансов
источник

ОН

Олег Неумывакин in QA Сибирь
Оно просто должно быть. Нет никакого "впечатления". Нет, код не станет всегда правильным и хорошим. Но все равно, нет никакой причины этого не делать.
источник

S

Sergei in QA Сибирь
Есть ещё такая вещь как культура. Она только с опытом приходит. Когда люди научатся делать ревью не на каждый мелкий коммит (постоянно отвлекая коллег), но и не будут выкатывать километровые ченжи которые посмотреть за вменяемое время нереально. Это только с опытом.
источник

ОН

Олег Неумывакин in QA Сибирь
Да, полностью согласен с Сергеем. И "мелкая" задача и "маленькое" изменение - это не агрумент чтобы не делать ревью, потому что один человек очень легко, в рамках своей маленькой задачи, может случайно закомитить что-то левое.
источник

ОН

Олег Неумывакин in QA Сибирь
Другое дело юнит-тесты, тут нельзя быть категоричным и нужно решать индивидуально.
источник

ОН

Олег Неумывакин in QA Сибирь
Тут же еще может быть так что юнит тесты то есть, но они не запускаются =)
источник

ОН

Олег Неумывакин in QA Сибирь
а если запускаются то их результаты не учитываются =)
источник