Skip to content

Функциональность установки меток (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

Merged
merged 5 commits into from
Aug 24, 2017

Conversation

khorevaa
Copy link
Member

@khorevaa khorevaa commented May 11, 2017

Добавлена функциональность установки меток (tags) при изменении версии конфигурации в хранилище.
PR Содержит изменения #67
п.с. реальный коммит изменений только 1.

@khorevaa khorevaa changed the title Добавлена функциональность установки меток (tags) при изменении версии конфигруации в хранилище Добавлена функциональность установки меток (tags) при изменении версии конфигурации в хранилище May 11, 2017
@khorevaa khorevaa changed the title Добавлена функциональность установки меток (tags) при изменении версии конфигурации в хранилище Функциональность установки меток (tags) при изменении версии конфигурации в хранилище May 11, 2017
@@ -149,10 +167,13 @@
Формат,
КоличествоКоммитовДоPush,
URLРепозитория,
Лимит);
Лимит,
ПрерватьВыполнениеБезКомментарияКВерсии,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вот тут уже скоро начнет проситься рефакторинг. Параметров все больше и больше...

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Да просится отдельный Класс по примеру gitrunner

@EvilBeaver
Copy link
Member

@khorevaa твои PR друг с другом связаны, и, к сожалению, долго лежат. Есть какой-то порядок их правильного вливания у тебя?

@khorevaa
Copy link
Member Author

Да сначала мержить PR #67 потом этот.

@khorevaa
Copy link
Member Author

Up @artbear @nixel2007

@nixel2007
Copy link
Member

@khorevaa можешь сделать ребейз?

Исправление ошибки резрешения конфликта
@khorevaa
Copy link
Member Author

khorevaa commented Aug 24, 2017

Устранил конфликты и правил свой же баг.

Добавлен потеряный код.
@nixel2007 nixel2007 merged commit fc3d78c into oscript-library:develop Aug 24, 2017
Copy link
Member

@artbear artbear left a 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 @@
,
,
Лимит,
ПрерватьВыполнениеБезКомментарияКВерсии);
ПрерватьВыполнениеБезКомментарияКВерсии,
,
Copy link
Member

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Репозитория,
Лимит,
ПрерватьВыполнениеБезКомментарияКВерсии);
ПрерватьВыполнениеБезКомментарияКВерсии,
ИмяВетки,
Copy link
Member

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Репозитория, ИмяВетки, ОтправитьНовыеМетки);
Copy link
Member

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,
Знач ПрерватьВыполнениеБезКомментарияКВерсии) Экспорт
Знач ПрерватьВыполнениеБезКомментарияКВерсии,
Знач ИмяВетки = Неопределено,
Copy link
Member

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Репозитория,
Лимит,
ПрерватьВыполнениеБезКомментарияКВерсии);
ПрерватьВыполнениеБезКомментарияКВерсии,
ИмяВетки,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

И здесь передается новый параметр ИмяВетки !
в PR ничего об этом не сказано :(

@artbear artbear modified the milestone: 2.2 Oct 10, 2017
@khorevaa khorevaa deleted the feature/auto-tags branch November 9, 2017 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants