никогда после пула мастера не ломалось, если в мастере не сломано.

Пару раз мастер начинал фейлится из-за дат в тестах

Не указали возможность разбега

Но тогда он и в бранчах фейлился

пока из того что мы узнали (подправь если в моих предположениях по цифрам сильные неточности).... ~16 PR с человека, которые могут жить неделю, это значит что пока висит 1 PR чувак уже запостит еще 3 с такими темпами. То есть смещение от "сделал задачу" до "влито" в среднем примерно неделя. Узкое место процесса - CTO который устраивает код ревью каждого PR и отлавливает там баги (что странно - почему они там есть?) Я так понимаю что до момента "сделал задачу" тоже может какое-то время пройти? По приведенным тобой числам это где-то 2-3 дня в среднем и не больше. Это в целом неплохо. Это значит что любые проблемы с код ревью, которые выявит ваш CTO можно пофиксить весьма быстро. Так же, в силу того что у вас много разработчиков и хорошее покрытие тестами, тесты всегда зеленые на момент попадания в мастер, смысла в PR нет. Ваш CTO может постфактум ревьювить работу за вчера прямо в мастере и чуть что говорить что бы подправили. Поскольку все работают в мастере - подправить занимает минуту. Меньше времени на переключение контекста. Более того, если у нас в работе столько фич (выходит что у тебя где-то ~20 фич постоянно висят незамердженными), фидбэк о них вместе мы получим только через какое-то время, скажем неделю. Что подразумевает необходимость довольно сильной проработки фич на момент TODO.

но я не верю в твои слова про маленькое количество конфликтов

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

но я не верю в твои слова про маленькое количество конфликтов

эт они не со мной на проекте тусят просто

где я могу взять и переместить все к из одного неймспейса в другой

я так понял пилишь ты https://www.customer-alliance.com/

или пока буду править багу отрефакторить половину модуля)

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

Щаче всего рефакторинг - это отдельная таска. И фичи - отдельные таски. И баги - отдельные таски.

Щаче всего рефакторинг - это отдельная таска. И фичи - отдельные таски. И баги - отдельные таски.

то есть вы все планируете

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

Да.

сколько процентов времени отводится на планирование?

Такие случаи СТО и отлавливает часто.

Плюс он тот, кто знает систему полностью, так как писал с нуля ее.

Один день в начале спринта.

Каждый второй понедельник мы часов 6 сидим эстимейтим и планируем

Плюс он тот, кто знает систему полностью, так как писал с нуля ее.

ну я был на его месте - так себе с точки зрения рисков по проекту)

Плюс он тот, кто знает систему полностью, так как писал с нуля ее.

знает всю систему

какая по размерам система то?

ну я был на его месте - так себе с точки зрения рисков по проекту)

Ну кстати благодаря его ревью у нас очень мало критических багов. Сильно мало.

какая по размерам система то?

А размер как сказать? В килограммах?

А размер как сказать? В килограммах?

можно loc

можно src/ померять loc

100K, 200K, 300K

А что такое ioc? :)

Line of Code

Аааа

loc :)

lines of code

Ща

lies of code(

В гитхабе это можно сделать?

Я в виндоусе сейчас дома просто

- ты где? - в винде!

:D

В гитхабе это можно сделать?

не... там максимум code frequency можно глянуть

тоже хорошая метрика

Упс. Дату обрезал

Ща еще раз

очень любопытные пики на 500K инсертов)

а эти взрывы что то сгенерированное кажись ))

Я думаю, это когда сильно систему переписывали.

у меня есть такие - это когда закоммитили node_modules

Я думаю, это когда сильно систему переписывали.

врядли

или webpack build

или webpack build

тоже да

Ребята. Не пытайтесь найти в системе мудака. :) Нет его.

Это нормальная добротная система.

Все что надо вигноре, все что надо где надо лежит

Это нормальная добротная система.

агрегатор ревью для анализа? я правильно понимаю?

Никакой херней не занимаемся

? нет сомнений

агрегатор ревью для анализа? я правильно понимаю?

Не понял вопроса.

Не понял вопроса.

пилите что

что за система - я вижу только то что у тебя на гитхабе

А да.

Это приложение сбором ревью не занимается.

Краулеры на скале.

паблик борда для пользователя?

Не паблик. Это Б2б

если в проекте 500к кодовая база, и CTO знает ее вдоль и поперек, он че только ревью занимается?

ERROR: S client not available

Ну мы конечно и API и виджет на сайт предоставляем

если в проекте 500к кодовая база, и CTO знает ее вдоль и поперек, он че только ревью занимается?

Он очень эффективный сука :)

у нас 500к+ кода, но это так дохрена что мы можем не знать что происходит у соседей в коде

если в проекте 500к кодовая база, и CTO знает ее вдоль и поперек, он че только ревью занимается?

это не кодовая база, это например за неделю 10 раз закоммитили cs-fixer

Он один из частых контрибьюторов в симфони

как зовут?

а дай его гитхаб глянуть)

dmaicher

ироничные PR: [TwigBridge] fix merge of 3.4 into master

dmaicher

?

как-то последние 3 месяца он не особо контрибьютил в симфони)

как-то последние 3 месяца он не особо контрибьютил в симфони)

не ну профайл норм

Ну он чаще когда баги какие-то вылазят, коммитит.

Так просто не сидит уж :)

ну короч не, давай данные по LoC

Причем баги, которые нас прямо касаются.

если там больше 200-300 LoC то маловероятно что он знает всю систему. Больше вероятно что у вас система декомпозирована нормально и нет нужды знать все

Как же дать-то вам их? :)

Ну он часто знает что откуда идет куда.

Уж не 100% всегда знает.

ну короч фичабрэнчи это рак

я все)

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

Ок :)

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

мне вот интересно какой процент времени он на это тратит и почему он вынужден это делать

а вообще глянуть бы код глазком одним... эх

фичабренч неделя прям с понедельник тема поднимается )))

Ответственный очень. Я это минусом считаю конечно тоже. Как и он. Но он боится, что пройдет мимо него что-то.

фичабренч неделя прям с понедельник тема поднимается )))

сказали ж хватит топить за сеттеры)

вот и начались фичабренчи

бгг

?