-
Notifications
You must be signed in to change notification settings - Fork 93
Функциональность установки меток (tags) при изменении версии конфигурации в хранилище #68
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Функциональность установки меток (tags) при изменении версии конфигурации в хранилище #68
Conversation
…и конфигруации в хранилище
src/core/Классы/КомандаSync.os
Outdated
@@ -149,10 +167,13 @@ | |||
Формат, | |||
КоличествоКоммитовДоPush, | |||
URLРепозитория, | |||
Лимит); | |||
Лимит, | |||
ПрерватьВыполнениеБезКомментарияКВерсии, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Вот тут уже скоро начнет проситься рефакторинг. Параметров все больше и больше...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Да просится отдельный Класс по примеру gitrunner
@khorevaa твои PR друг с другом связаны, и, к сожалению, долго лежат. Есть какой-то порядок их правильного вливания у тебя? |
Да сначала мержить PR #67 потом этот. |
@khorevaa можешь сделать ребейз? |
Исправление ошибки резрешения конфликта
Устранил конфликты и правил свой же баг. |
Добавлен потеряный код.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nixel2007 @khorevaa Смешались изменения из другого PR ? :(
@@ -99,7 +108,9 @@ | |||
, | |||
, | |||
Лимит, | |||
ПрерватьВыполнениеБезКомментарияКВерсии); | |||
ПрерватьВыполнениеБезКомментарияКВерсии, | |||
, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Что за новый параметр, передаваемый через запятую? Ветка?
в PR ничего об этом не сказано :(
@@ -159,10 +169,12 @@ | |||
КоличествоКоммитовДоPush, | |||
URLРепозитория, | |||
Лимит, | |||
ПрерватьВыполнениеБезКомментарияКВерсии); | |||
ПрерватьВыполнениеБезКомментарияКВерсии, | |||
ИмяВетки, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
И здесь передается новый параметр ИмяВетки !
в PR ничего об этом не сказано :(
СчетчикКоммитов = СчетчикКоммитов + 1; | ||
|
||
Если ИспользоватьПромежуточныйPUSH И | ||
СчетчикКоммитов = КоличествоКоммитовДоPush Тогда | ||
ВыполнитьGitPush(КаталогРабочейКопии, URLРепозитория); | ||
ВыполнитьGitPush(КаталогРабочейКопии, URLРепозитория, ИмяВетки, ОтправитьНовыеМетки); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
И здесь передается новый параметр ИмяВетки !
в PR ничего об этом не сказано :(
@@ -79,7 +79,9 @@ | |||
Знач КоличествоКоммитовДоPush = 0, | |||
Знач URLРепозитория = Неопределено, | |||
Знач Лимит = 0, | |||
Знач ПрерватьВыполнениеБезКомментарияКВерсии) Экспорт | |||
Знач ПрерватьВыполнениеБезКомментарияКВерсии, | |||
Знач ИмяВетки = Неопределено, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
И здесь передается новый параметр ИмяВетки !
в PR ничего об этом не сказано :(
@@ -90,7 +92,9 @@ | |||
КоличествоКоммитовДоPush, | |||
URLРепозитория, | |||
Лимит, | |||
ПрерватьВыполнениеБезКомментарияКВерсии); | |||
ПрерватьВыполнениеБезКомментарияКВерсии, | |||
ИмяВетки, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
И здесь передается новый параметр ИмяВетки !
в PR ничего об этом не сказано :(
Добавлена функциональность установки меток (tags) при изменении версии конфигурации в хранилище.
PR Содержит изменения #67
п.с. реальный коммит изменений только 1.