These are chat archives for silverbulleters/vanessa-behavior

14th
Sep 2016
Sergey Novikov
@snov1kov
Sep 14 2016 05:15
@Pr-Mex Доброе утро, после вчерашнего обновления Ванессы перестало строится дерево фич. Выдает ошибку при наличии повторных сниппетов. Надо удалять дубли теперь? Раньше обработка ругалась, но дерево успешно строилось
Sergey Novikov
@snov1kov
Sep 14 2016 05:30
Плохо то, что если один из разработчиков добавит по неосмотрению повторный сниппет, падает вся сборка (но при этом в Дженкинсе светится зеленым)
Evhen Sosna
@pumbaEO
Sep 14 2016 06:52
Имхо согласен так падать не должно, и если встречаются сниппеты похожие, то берется первый по приоритету
Pautov Leonid
@Pr-Mex
Sep 14 2016 06:56
@snov1kov этот код был и раньше
самоё весёлое что он не работал раньше из-за ошибки
а вчера я её поправил
я считаю ситуацию дублирования снипетов исключительной
это означает, что у нас есть две разные процедуры, и какая из них правильная - не понятно
поэтому выдаётся исключение - чтобы был сигнал что надо навести порядок
@snov1kov у вас реально есть дублирование снипетов?
git22ura
@git22ura
Sep 14 2016 06:58

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

А где про приоритеты можно глянуть?

Evhen Sosna
@pumbaEO
Sep 14 2016 06:58
конечно, есть библиотека UI ванессы и моя, где сниппеты определенные повторяются, моя первая стоит в списке библиотек.
т.е. в списке библиотек сначала мои библиотеки потом ванессы подключаются и вот если дублируется сниппет библиотечный, то он должен браться первый по списку.
Pautov Leonid
@Pr-Mex
Sep 14 2016 06:59
понял тебя
Sergey Novikov
@snov1kov
Sep 14 2016 07:00
@Pr-Mex да, у нас реально есть дубли, пытаюсь с этим бороться
Pautov Leonid
@Pr-Mex
Sep 14 2016 07:04
давайте тогда сюда перейдём
silverbulleters/vanessa-behavior#44
надо формально правила описать и затвердить
Sergey Novikov
@snov1kov
Sep 14 2016 07:11
@Pr-Mex нам бы пока хотфикс, что бы сборки не ломались
Сообщение вместо Вызвать исключение
Pautov Leonid
@Pr-Mex
Sep 14 2016 07:13
я обновление могу выпустить только вечером
откатитесь пока назад на пару коммитов
Pautov Leonid
@Pr-Mex
Sep 14 2016 07:48
сегодня заменю Исключение на Сообщить, пока не принято решение по приоритету снипетов
Sergey Novikov
@snov1kov
Sep 14 2016 08:41
@Pr-Mex спасибо