Добавил выгрузку с учетом 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
| @@ -8,7 +8,7 @@ | |||
| .ЗависитОт("cmdline", "0.4.1") | |||
| .ЗависитОт("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 версии!
| Иначе | ||
| МассивФайлов = НайтиФайлы(КаталогВыгрузки, "*.bin", Истина); | ||
| Для каждого Файл из МассивФайлов Цикл | ||
| Если Нрег(Прав(Файл.ПолноеИмя, 5)) = ".form" Или Нрег(Файл.Имя) = "form.bin" Тогда |
There was a problem hiding this comment.
@pumbaEO Я недавно делал аналогичное замечание - условие на .form никогда не юзается, потому что ищем-то *.bin
Нужно убрать это условие
| @@ -403,7 +439,7 @@ | |||
|
|
|||
| КонецПроцедуры | |||
|
|
|||
| Процедура РаспаковатьКонтейнерМетаданных(Знач ФайлРаспаковки, Знач КаталогРаспаковки, Знач Переименования, Знач КорневойКаталог) | |||
| Процедура РаспаковатьКонтейнерМетаданных(Знач ФайлРаспаковки, Знач КаталогРаспаковки, Знач Переименования = Неопределено, Знач КорневойКаталог = Неопределено) | |||
There was a problem hiding this comment.
А почему Знач КорневойКаталог = Неопределено ?
Это же строка в чистом виде.
1 Предлагаю Знач КорневойКаталог = ""
2 Добавить утверждение - корневой каталог не должен быть пуст, если Переименования <> Неопределено
- иначе это бессмысленно
| @@ -911,7 +948,8 @@ | |||
| Знач ПрерватьВыполнениеБезКомментарияКВерсии = Ложь, | |||
| Знач ИмяВетки = Неопределено, | |||
| Знач АвтоматическаяУстановкаТэговПоВерсиям = Ложь, | |||
| Знач ПроверитьАвторовХранилища = Ложь) Экспорт | |||
| Знач ПроверитьАвторовХранилища = Ложь, | |||
There was a problem hiding this comment.
Ребята, в методе СинхронизироватьХранилищеКонфигурацийСГит уже 13 параметров!!
Контрибьюторы, Неужели не болит душа при добавлении очередного параметра??
| @@ -911,7 +948,8 @@ | |||
| Знач ПрерватьВыполнениеБезКомментарияКВерсии = Ложь, | |||
| Знач ИмяВетки = Неопределено, | |||
| Знач АвтоматическаяУстановкаТэговПоВерсиям = Ложь, | |||
| Знач ПроверитьАвторовХранилища = Ложь) Экспорт | |||
| Знач ПроверитьАвторовХранилища = Ложь, | |||
| Знач ТолькоИзменения = Ложь) Экспорт | |||
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.
Ну я шел от обратного, а потом оказалось игла в яйце, яйцо в ... и т.д.
| @@ -1431,3 +1474,4 @@ | |||
| ДоменПочтыДляGitПоУмолчанию = "localhost"; | |||
| УдалятьВременныеФайлы = Ложь; | |||
| КоличествоЦикловОжиданияЛицензииПоУмолчанию = 1; | |||
| ТолькоИзменения = Ложь; | |||
There was a problem hiding this comment.
Добавить перевод строки для исключения проблем с Гитом
|
@pumbaEO Если ты не закончил, добавь |
|
100 версий за 52 минуты, на той же конфе, раньше за час 10 версий синхронизировалось. @artbear я закончил, 2 проекта у себя уже синхронизировал. |
|
Отлично! |
|
@pumbaEO В доку добавишь изменения? |
Первая попытка #29 добавления инкрементальной выгрузки.