Eoksni
commented
7 years ago Хрен с ним, пусть так, не надо ничего менять, но напоминаю workflow на котором мы вроде сошлись:
- Под каждую версию - своя ветка. В данном случае должна была быть ветка v0.2
- Под каждый таск (или группу схожих тасков) - своя ветка. В данном случае под е2е таски должна быть ветка e2e-tests и под таск на ридми - еще ветка readme.
- Под каждую из веток (e2e-tests и readme) делаешь пулл реквест, где в качестве base указываешь v0.2 (то есть замержены эти ветки будут в v0.2).
- После того как каждый таск отдельно заревьювен и смержен в v0.2, делаешь общий пулл реквест на всю версию v0.2 в мастер.
Напоминаю зачем это надо:
- Значительно удобнее все это дело ревьювить и обсуждать, когда под каждый таск или группу схожих тасков отдельный пулл реквест.
- Зачем вообще заморачиваться с ветками версий и почему не мержить ветки тасков сразу в мастер - потому что ревью отдельного таска более "халявное", не включает в себя прогон тестов и просмотр всего проекта в целом и его тестирование, ибо шибко это дело трудоемкое. Между тем время от времени это делать все же нужно, и один полноценный ревью на версию - самое то.
В данном случае у тебя был по сути один таск на е2е тесты, и сбоку к нему мелочь по ридми, так что подобный workflow это несколько перебор, В результате ты все сделал ровно по тому же принципу что в пред версии v0.1, только ветку назвал e2e-tests, а не v0.2. Короче лажа какая-то получилась :) Давай все же на след версии попробуем сделать по новому workflow.
ITarako
Все готово, за одним исключением, в доке нет описания установки вебдрайвера, я морально не готов к этому геморою