Замена использования bat при работе с git #69
Conversation
|
Oops! Законфликтилось. @khorevaa разрули, плиз. |
|
Сливать PR рано. Жду когда с gitrunner будут приняты другие и PR и передалаю gitrunner на работы с 1commands |
|
@khorevaa наверное надо еще сам gitrunner перевести на 1commands, иначе будет зависание чтения на потоках. |
|
туплю, он уже на 1commands |
|
Хм... оно даже собралось... |
|
@nixel2007 так я вроде gitrunner и переводил.. |
artbear
left a comment
artbear
left a comment
There was a problem hiding this comment.
@khorevaa @nixel2007 Нужны доработки или ответы на вопросы!
Жду
There was a problem hiding this comment.
Метод ПолучитьГитРепозиторий не корректно вызывается
сам метод часто вызывается перед вызовом команд гита (пуш, коммит и т.п.)
а при вызове этот метод постоянно вызывает установку настроек, что совершенно не нужно для указанных команд :(
Предлагаю добавить установку этих параметров один раз в начале работы продукта, а в дальнейшем уже не устанавливать настройки.
@khorevaa @nixel2007
Функция ПолучитьГитРепозиторий(Знач КаталогРабочейКопии)
- Если Дата = Неопределено Тогда
- Дата = ТекущаяДата();
- КонецЕсли;
+ ГитРепозиторий = Новый ГитРепозиторий;
+ ГитРепозиторий.УстановитьРабочийКаталог(КаталогРабочейКопии);
+ ГитРепозиторий.УстановитьНастройку("core.quotepath","false",РежимУстановкиНастроекGit.Локально);
+ ГитРепозиторий.УстановитьНастройку("merge.ours.driver","true",РежимУстановкиНастроекGit.Локально);
+ Возврат ГитРепозиторий;
+
+КонецФункции // ПолучитьГитРепозиторий()There was a problem hiding this comment.
@khorevaa @nixel2007 Соответствующая веселуха с ci-сервера проекта :(
дофига ненужных команд
There was a problem hiding this comment.
А не надо включать отладочные логи всех библиотек :Р
There was a problem hiding this comment.
Странные логи. Видимо из gitrunner. Надо посмотреть почему лезит
There was a problem hiding this comment.
потому что я это логирую, очевидно. вангую, что Артур как обычно включил все логи через rootLogger. естественно он видит всею
There was a problem hiding this comment.
тут проблема не в лишних логах, а в том, что вызывается лишние и абсолютно не нужные команды, о которых я уже писал выше.
There was a problem hiding this comment.
+ ГитРепозиторий.УстановитьРабочийКаталог(КаталогРабочейКопии);
+ ГитРепозиторий.УстановитьНастройку("core.quotepath","false",РежимУстановкиНастроекGit.Локально);
+ ГитРепозиторий.УстановитьНастройку("merge.ours.driver","true",РежимУстановкиНастроекGit.Локально);There was a problem hiding this comment.
Как раз в данном случае отладочные логи очень четко показывают проблему.
There was a problem hiding this comment.
Лог.Отладка(СтрШаблон давно устарел.
СтрШаблон не нужен, можно сразу писать Лог.Отладка("%1", Парам);
There was a problem hiding this comment.
Лог.Отладка(СтрШаблон давно устарел.
СтрШаблон не нужен, можно сразу писать Лог.Отладка("%1", Парам);
There was a problem hiding this comment.
Лог.Отладка(СтрШаблон давно устарел.
СтрШаблон не нужен, можно сразу писать Лог.Отладка("%1", Парам);
There was a problem hiding this comment.
Лог.Отладка(СтрШаблон давно устарел.
СтрШаблон не нужен, можно сразу писать Лог.Отладка("%1", Парам);
There was a problem hiding this comment.
Лог.Отладка(СтрШаблон давно устарел.
СтрШаблон не нужен, можно сразу писать Лог.Отладка("%1", Парам);
There was a problem hiding this comment.
Лог.Отладка(СтрШаблон давно устарел.
СтрШаблон не нужен, можно сразу писать Лог.Отладка("%1", Парам);
There was a problem hiding this comment.
Так и ПР висел хз скок времени
There was a problem hiding this comment.
@khorevaa @nixel2007 Почему подменяется возврат результата?
Было
Возврат Результат;
Стало
Возврат 0;
There was a problem hiding this comment.
Ммм косяк. Надо подумать! Поправлю на досуге хотя там вылетить исключение уже из gitrunner
There was a problem hiding this comment.
@khorevaa @nixel2007 Почему подменяется возврат результата?
Было
Возврат Результат;
Стало
Возврат 0;
4 participants
Произведена замена основного функциаонала по работе с GIT. Теперь используется библиотека gitrunner.
Использование в тестах не заменено.
Важно!! Сливать PR можно только после выхода новой версии (предположительно 1.1.0) gitrunner (https://github.com/nixel2007/gitrunner)
Для проверки необходимо установить gitrunner из ветки develop