Добавил выгрузку с учетом increment #90
Conversation
|
Не знаю почему написало ошибку, локально у меня все прошло. |
There was a problem hiding this comment.
Нужны исправления.
Ждем прохождения сборки, которую я вручную запустил - http://build.oscript.io/job/oscript-library/job/gitsync/job/PR-90/4/console
packagedef
Outdated
| .ЗависитОт("tempfiles") | ||
| .ЗависитОт("tool1cd", "0.4") | ||
| .ЗависитОт("v8runner", "0.7.0") | ||
| .ЗависитОт("v8runner") |
There was a problem hiding this comment.
Почему убрана зависимость от конкретной версии v8runner ?
There was a problem hiding this comment.
Предлагаю добавить зависимость от последней версии v8runner
There was a problem hiding this comment.
Та ну..., что-бы потом забыть об этой зависимости, как вот ниже ("1commands", "1.1.1") а gitrunner уже давно 1.5 версии!
There was a problem hiding this comment.
@pumbaEO Я недавно делал аналогичное замечание - условие на .form никогда не юзается, потому что ищем-то *.bin
Нужно убрать это условие
There was a problem hiding this comment.
А почему Знач КорневойКаталог = Неопределено ?
Это же строка в чистом виде.
1 Предлагаю Знач КорневойКаталог = ""
2 Добавить утверждение - корневой каталог не должен быть пуст, если Переименования <> Неопределено
- иначе это бессмысленно
There was a problem hiding this comment.
Ребята, в методе СинхронизироватьХранилищеКонфигурацийСГит уже 13 параметров!!
Контрибьюторы, Неужели не болит душа при добавлении очередного параметра??
There was a problem hiding this comment.
@pumbaEO Тут путаница с одинаковым названием и глобальной переменной, и параметра метода.
Для разруливания этой путаницы тебе пришлось добавить аж спец.метод УстановитьРежимТолькоОбновленияФайлов, который вызывается всего один раз, состоит из одной строки :(
Предлагаю
-
либо параметр метода либо глобальную переменную назвать
ВыгружатьТолькоИзменения, а вторуюТолькоИзменения(или наоборот) -
А метод
УстановитьРежимТолькоОбновленияФайловудалить
There was a problem hiding this comment.
Ну вот как раз хотел спец метод добавить, что-бы не кидать в параметр + 1
There was a problem hiding this comment.
Так 13-й параметр-то остался :(
т.е. метод создан, но никто его не вызывает?
There was a problem hiding this comment.
Ну я шел от обратного, а потом оказалось игла в яйце, яйцо в ... и т.д.
There was a problem hiding this comment.
Добавить перевод строки для исключения проблем с Гитом
|
@pumbaEO Если ты не закончил, добавь |
|
100 версий за 52 минуты, на той же конфе, раньше за час 10 версий синхронизировалось. @artbear я закончил, 2 проекта у себя уже синхронизировал. |
|
Отлично! |
|
@pumbaEO В доку добавишь изменения? |
2 participants
Первая попытка #29 добавления инкрементальной выгрузки.