@symfony_php

Страница 590 из 1418
Sergey
22.01.2018
22:48:17
Alex
22.01.2018
22:48:28
И получается, что когда все шпилят в WIP кто то один может заморозить движение в прод

Sergey
22.01.2018
22:48:34
ну то есть, что значит "налажал с WIP"?

Alex
22.01.2018
22:48:44
))) Есть mergers

Google
Sergey
22.01.2018
22:48:52
и никаких фризов релиза

Alex
22.01.2018
22:49:27
ну вот WIP не идет в зеленый туда пушило 10 разрабов

из за кого?

Sergey
22.01.2018
22:49:58
это как раз таки легко затрекать - чей коммит тот и папа

у тебя ж билд не выборочно срабатывает. 10 человек запушило - 10 раз был билд

Alex
22.01.2018
22:51:00
:D таак

билд все таки делается по каждому коммиту?

Sergey
22.01.2018
22:51:43
билд все таки делается по каждому коммиту?
ну типа маловероятно что у тебя все 10 человек решили запушить в одну и ту же секунду

Alex
22.01.2018
22:51:48
а если после фэйл билда пошли дальше коммиты?

Sergey
22.01.2018
22:52:16
а если после фэйл билда пошли дальше коммиты?
не пойдут, это одино из правил CI, если билд красный - у команды нет приоритета выше чем сделать его зеленым.

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

Alex
22.01.2018
22:52:46
есть выдающиеся пушеры которые в вип пушат так что кофе не успеешь допить ?

Google
Sergey
22.01.2018
22:53:17
Жестоко
на самом деле простой git revert все пофиксит если тебе лень)

Alex
22.01.2018
22:53:18
но все таки применимо для работы

Sergey
22.01.2018
22:53:47
так что тебе надо просто дождаться окончания сборки и статуса

Alex
22.01.2018
22:54:01
Почетал ветку и звучало оно как анархия

согласен, применимо для работы

Sergey
22.01.2018
22:54:29
более того - намного эффективнее флоу с фичабрэнчами в силу кучи факторов. Но может не сработать в силу специфики модели доставки

ну или как у тебя - опенсурс - тут только через PR, во всяком случае для недоверенных лиц

Alex
22.01.2018
22:55:43
ну как бы если проект даже частично опен сурс то не катит а для рабочих закрытых процессов да

Sergey
22.01.2018
22:55:57
не вижу препядствий

Alex
22.01.2018
22:56:05
?

это к народу с OroCommerce

те не совсем по осс протоколу внутри работают а снаружи через PR

...вот и поговорили )))) можно вернуться к теме типов, постфиксной форме и котлину ?

Sergey
22.01.2018
22:59:01
Alex
22.01.2018
22:59:52
Ocramius одобряет котлин



Google
Evgen
22.01.2018
23:10:51
имхо СД это то о чем так долго говрили большивики...

но для этого надо сначала CI полноценный и покрытие кода тестами увеличить (end-to-end для начала намашнить)

каждый в это понятие вкладывает все что угодно, иногда волос дыбом встает) после общение с кандидатами для галочки приписывающие "CD" в резюме

1. так если по существу, СД без тестов = утопия. 2. бизнесс не дает на тесты ресурс - ваши проблемы (с) 3. мы не пишим тесты потому что за них никогда не платят, смотри п 2. 4. не помогает п.3 - есть варианты

Alex
22.01.2018
23:24:29
?

Andy
22.01.2018
23:35:40
Вечер в радость, коммитик в сладость, сенсиолабс по масти, формошлепам по пасти. Уважаемые сидельцы (за компом) , обращаюсь к вам с вопросом, али не в напряг - просветите молодого гребца. Коли вы встречались с рантайм-парсингом не всегда валидного хтмл под большой нагрузкой, то что использовали (технологии, вендоры, алгоритмы). Зрение премного благодарен

Combot
22.01.2018
23:38:22
combot.org/chat/-1001091570538

Sergey
23.01.2018
07:17:39
логгер в конструктор
но ведь он не несет никакой логики и занимает целое место в зависимостях) и мокать его не нужно

я к примеру предпочитаю \Psr\Log\LoggerAwareTrait + new NullLogger в конструкторе

Dmitriy
23.01.2018
07:37:28
Я еще юзаю евенты для этого

чтобы сервис с бл не зависел от инфраструктуры

получается сервис кидает евент, а уже логгер-подписчик логирует как ему надо

Sergey
23.01.2018
07:41:19
слишком много лишних телодвижений получается

смотря что еще за логи

если это какой-нибудь TraceableEventDispatcher то тут вообще декораторами делают

а если просто в монолог пописать то почему бы и в самом сервисе не использовать?

Dmitriy
23.01.2018
07:46:20
я щитаю нарушение SRP

если идти дальше то "а чо бы и письмо не отправить" )

Sergey
23.01.2018
07:47:03
в джаве логгеры к примеру вообще из статической фабрики берутся

и сами логгеры это статическое поле в классе

Google
Sergey
23.01.2018
08:28:09
как бы есть случаи когда народ увольняли за написание тестов)))) ?
очень интересно было бы поговорить с обеими сторонами сего конфликта

Dmitriy
23.01.2018
08:28:38
Sergey
23.01.2018
08:28:47
тут не так давно проходил чувак которого уволили за рефакторинг.... и в том случае работодатель поступил правильно потому что... ну там все плохо было)

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

просто надо было тесты писать втихаря )
ну тип того. Если ты сабтаск на тесты делаешь - значит ты уже проиграл и подсознательно не хочешь их писать

Admin
ERROR: S client not available

Sergey
23.01.2018
08:31:31
я щитаю нарушение SRP
я не считаю это нарушением SRP, чаще это нарушение open/close

Alex
23.01.2018
08:31:34
??? втихаря вариант но зависит от того кто сверху наблюдает за процессом и как шарит...

Sergey
23.01.2018
08:32:16
??? втихаря вариант но зависит от того кто сверху наблюдает за процессом и как шарит...
если ты вообще заморачиваешься такими вопросами - значит тесты у тебя не окупаются, а это надо думать почему.

тесты (особенно юнит) не должны налагать сильно много оверхэда на время разработки (ну может 10% каких, но не больше). e2e посложнее, но тут надо инфраструктуру один раз нормально сделать и потом не больно.

ну и конечно все обычно фэйлятся на этапе "я только учусь" когда оверхэд (как и от любой новой фигни) может и 200% достигать

Dmitriy
23.01.2018
08:33:59
да юниты даже для себя полезно будет, автоматизировать тестирование

чтобы не "тыкать" одно и тоже по сто раз

Bohdan
23.01.2018
08:36:43
кстати tdd на юнитах - это норм или бред, который я сам выдумал?

Sergey
23.01.2018
08:36:52
а как еще?)

Dmitriy
23.01.2018
08:37:02
почему бред?

Bohdan
23.01.2018
08:37:11
на e2e

Google
Bohdan
23.01.2018
08:37:21
ну или это bdd и я путаю понятия

Sergey
23.01.2018
08:44:52
когда не на юнитах - это atdd (acceptance test driven)

Alex
23.01.2018
08:45:03
tdd на юнитах это то самое

Bohdan
23.01.2018
08:45:17
ага, спасибо

Sergey
23.01.2018
08:45:23
Alex
23.01.2018
08:45:54
behat и прочая е2е

Sergey
23.01.2018
08:46:09
ну или это bdd и я путаю понятия
bdd это вообще не про тесты. ну то есть как, оно пересекается с acceptance тестами но это больше про разговоры и формализацию требований

ну то есть bdd это specification by example + atdd

Bohdan
23.01.2018
08:46:48
ну да, с учетом того, что там де-факто стандарт behat, который словесно почти все описывает...

Alex
23.01.2018
08:47:15
ну sylius к примеру не тестит?

они кажись все через безат гоняют

Sergey
23.01.2018
08:48:02
behat + phpspec

Alex
23.01.2018
08:48:03
словесность нужно наполнить словами

тьфу смыслом

Sergey
23.01.2018
08:48:14
@ui Scenario: Changing address of my order Given I had product "PHP T-Shirt" in the cart And I specified the shipping address as "Ankh Morpork", "Frost Alley", "90210", "United States" for "Jon Snow" When I go back to addressing step of the checkout And I change the shipping address to "Ankh Morpork", "Fire Alley", "90350", "United States" for "Jon Snow" And I complete the addressing step Then I should be on the checkout shipping step

это если что пример сценария из силиуса

Alex
23.01.2018
08:48:32
??

Sergey
23.01.2018
08:48:38
вполне себе bdd

Alex
23.01.2018
08:49:54
это же только геркин часть

Страница 590 из 1418