Доработки gitsync tags push tempdir authors branch #6

khorevaa · 2016-11-22T11:41:24Z

Добавлены следующие возможности:

Автоматическая установка и выгрузка tags
Возможность указания каталога temp
Возможность выполнение push - каждые n-коммитов
Возможность указания ветки (branch)
Изменен формат хранения AUTHORS на JSON, добавлена возможность конвертации и инициации через командную строку
Изменен принцип работы команды push: на использование "refs:/" вместо "--all"

…ый-n-версий-хранилища to master * commit '1a0a08b61b201b763fb81a89b23683bce0084ff0': Исправлен вызов процедуры синхронизации с новыми параметрами Добавление отправки на удаленный репо каждые n - коммитов

…параметра-ветки-в-gitsync-all # Conflicts: # src/multi-controller.os # src/xml-config.os

…-ветки-в-gitsync-all to master * commit 'e9aca5e974c44d326b7bb178b5d853faea05c290': Добавлена возможность указания ветки при пакетной синхронизации

…и-указания-расположения-рабочего-каталога to develop * commit '7d55345447bf90ae41ebbefb239cfdb4ed835fca': Исправлена ошибка преобразования типов Дорабавлен ключ "-tempcatalog" Добавлена возможность указания Temp каталога

* commit 'f2decf724a37db26a262ea684a4d941b3d9c7e72': Исправлена ошибка преобразования типов Дорабавлен ключ "-tempcatalog" Добавлена возможность указания Temp каталога

* commit 'b2e6edb60af6762b933d33ac92ef5e3150a181c9':

# Conflicts: # src/core/Классы/МенеджерСинхронизации.os

Добавлены консольные команды

* commit '706464c9de369447b4992c3688002eee7178c923': Очередная правка багов Исправлены баги. Добавлены консольные команды Переработка хранения файла AUTHORS Добавил Экспорт методу

* commit 'b260cc968ac0f1c587dc0d91cc1101859091c121': Очередная правка багов Исправлены баги. Добавлены консольные команды Переработка хранения файла AUTHORS Добавил Экспорт методу

* commit '9af89bfb6f043581cc6cd2ecc6a3d8ef50df32a3':

в tempdir

nixel2007 · 2016-11-22T12:32:33Z

.vscode/settings.json

@@ -0,0 +1,3 @@
+// Поместите параметры в этот файл, чтобы перезаписать параметры по умолчанию и пользовательские параметры.


Этот файл разве нужен в гите? Пустой по крайней мере.

Не нужен.

Да не корректный gitignore...

nixel2007 · 2016-11-22T12:34:28Z

src/core/Классы/МенеджерСинхронизации.os

@@ -24,6 +24,8 @@

 Перем ДоменПочтыДляGitПоУмолчанию Экспорт;
 Перем ВерсияПлатформы Экспорт;
+Перем мAUTHORS Экспорт;


Эти переменные точно экспортные? Обычно через м все же приватные добавляют

@nixel2007 не к той строчке прикрепил. Это экспортные. Это свойства объекта. Их добавлял я. А вот те, что ниже - да, твой вопрос актуален.

Хм... подумаю... видимо для консольной команды надо было..

nixel2007 · 2016-11-22T12:34:57Z

src/core/Классы/МенеджерСинхронизации.os

@@ -371,7 +373,7 @@

 // Выполняет фиксацию изменений в локальном каталоге git
 //
-Процедура ВыполнитьКоммитГит(Знач КаталогРабочейКопии, Знач Комментарий, Знач Автор, Знач Дата=Неопределено) Экспорт
+Процедура ВыполнитьКоммитГит(Знач КаталогРабочейКопии, Знач Комментарий, Знач Автор, Знач Дата=Неопределено, Знач УстановитьНовуюМетку, Знач Метка) Экспорт


Новые параметры без дефолтных значаний

Дополню. Зачем 2 параметра? Установить метку и сама метка. Почему не сделать только "Метка" и если она указана, то устанавливать ее?

Да верно... Не подумал. Исправлю! . :)

nixel2007 · 2016-11-22T12:35:10Z

src/core/Классы/МенеджерСинхронизации.os

@@ -386,6 +388,7 @@
 	КомандныйФайл = СоздатьКомандныйФайл();
 	ПрефиксЭкспортаПеременной = ?(ЭтоWindows, "set", "export");
 	Если ЭтоWindows Тогда
+		//ДобавитьВКомандныйФайл(КомандныйФайл, "chcp 866");


Удалить

Принято

nixel2007 · 2016-11-22T12:35:50Z

src/core/Классы/МенеджерСинхронизации.os

-	Если Найти(Автор, "<") <= Найти(Автор, ">") Тогда
-		авторДляГит = Автор+" <"+Автор+"@localhost>"; // e-mail может быть удобен для поиска в связанных системах //авторДляГит = Автор+" <"+Автор+">";
-	КонецЕсли;
+	Лог.Информация("Автор коммита: <"+Автор+">");


Эта информация нужна в логе?

Согласен, нужно перенести в отладочный лог.

Принято...

nixel2007 · 2016-11-22T12:38:59Z

src/core/Классы/МенеджерСинхронизации.os

+	КонецЕсли;	 
+
+
+	//ПараметрыКоманды.Добавить("--all -v");


Удалить

Принято

nixel2007 · 2016-11-22T12:40:15Z

src/core/Классы/МенеджерСинхронизации.os

@@ -478,6 +496,7 @@

 КонецФункции

+


Удалить

Принято.

nixel2007 · 2016-11-22T12:41:33Z

src/core/Классы/МенеджерСинхронизации.os


 КонецФункции

 // Считывает из хранилища историю коммитов с привязкой к пользователям
 //
-Функция ПрочитатьИзХранилищаИсториюКоммитовСАвторами(Знач ФайлХранилища) Экспорт
+Функция КонвертироватьТаблицуВерсийИзФорматаБД(Знач ТаблицаБД)


Такое чувство, что перенесли местами процедуры, но как-то не аккуратно

Ага, начиная с этого места, начинаются проблемы с код-ревью.
Гит просто показывается совершенно разные строки.

@khorevaa Предлагаю вернуть порядок функций, иначе сейчас очень-очень трудно выполнить полноценный код-ревью.

Порядок функций я не менял... просто почистил лишние и неиспользуемые..... как то так.. что с этим делать не знаю... вернуть старые функции?

@khorevaa нет, просто сделать изменения более мелкими.

nixel2007 · 2016-11-22T12:41:56Z

src/core/Классы/МенеджерСинхронизации.os

@@ -590,141 +609,411 @@

 	ЧтениеБазыДанных.ЗакрытьФайл();

-	Возврат КонвертироватьТаблицуПользователейИзФорматаБД(ТаблицаБД);
+	Возврат ТаблицаБД;


Так и надо?

Да все верно ковертация теперь не делается...

EvilBeaver · 2016-11-22T13:17:45Z

src/core/Классы/МенеджерСинхронизации.os


 	// КомандаКоммита = "git commit -a --file="""+ИмяФайлаКомментария+""" --author="""+Автор+""" >"+ИмяФайлаЛогаКоммита;
 	КомандаКоммита = "git commit -a --file="""+ИмяФайлаКомментария+""" --author="""+Автор+""" "+СуффиксПеренаправленияВывода(ИмяФайлаЛогаКоммита, Истина);
 	ДобавитьВКомандныйФайл(КомандныйФайл, КомандаКоммита);
+
+	Если УстановитьНовуюМетку Тогда


См. выше. 2 параметра не нужны.

Если ЗначениеЗаполнено(Метка) Тогда // устанавливаем метку

Принято поправлю....

artbear · 2016-11-22T13:18:31Z

src/core/Классы/МенеджерСинхронизации.os


 КонецФункции

 // Считывает из хранилища историю коммитов с привязкой к пользователям
 //
-Функция ПрочитатьИзХранилищаИсториюКоммитовСАвторами(Знач ФайлХранилища) Экспорт
+Функция КонвертироватьТаблицуВерсийИзФорматаБД(Знач ТаблицаБД)


Ага, начиная с этого места, начинаются проблемы с код-ревью.
Гит просто показывается совершенно разные строки.

@khorevaa Предлагаю вернуть порядок функций, иначе сейчас очень-очень трудно выполнить полноценный код-ревью.

EvilBeaver · 2016-11-22T13:18:57Z

src/core/Классы/МенеджерСинхронизации.os

@@ -458,7 +466,17 @@
 	ПараметрыКоманды = Новый Массив;
 	ПараметрыКоманды.Добавить("git push -u");
 	ПараметрыКоманды.Добавить(СтрЗаменить(УдаленныйРепозиторий, "%", "%%"));
-	ПараметрыКоманды.Добавить("--all -v");
+
+	СтрокаRefsHeads = "refs/heads/%1:refs/heads/%1";


Зачем вообще сей функционал? Какой сценарий использования?

Напомню: мы пытались добавить функционал веток и поняли, что он не нужен.

Функционал нужен для корректной отправки tags. Да и имя бранча мне нужно было т.к. выгружалось девелопорское хранилище.. (не грузить же его в мастер) .....

@khorevaa у вас всегда пуш/пулл делается в текущую ветку. Вот какую установили текущей, такая и синхронизируется. Явное указание ветки внутри gitsync потребует еще и команду checkout. А ее нет.

Т.е. сейчас вы например на ветке develop. Сделали синк с параметром ветки мастер, оно разложило, сделало пуш в мастер (но изменений для мастера нет, они выгрузились и закоммитились в текущий develop). В результате пуш будет пустой.

@EvilBeaver даже в режиме all?

EvilBeaver · 2016-11-22T13:21:05Z

src/core/Классы/МенеджерСинхронизации.os

+//////////////////
+// Работа с AUTHORS
+
+Процедура СконвертироватьAuthorsВФормалJSON(Знач ПутьКФайлуАвторов, Знач ДанныеUSERS) Экспорт


@khorevaa подскажи, для чего эта доработка?

Такой масштабный PR, я бы, если честно, порезал его на более мелкие, 1 функциональность на один пулреквест.

+1 за более мелкие PR.

С другой стороны, автору ИМХО будет достаточно тяжело выделить мелкие изменения :(

Да с выделением будет много проблем.. Но в принципе можно....

Да и решал я коллизии...

artbear · 2016-11-22T13:21:06Z

src/core/Классы/МенеджерСинхронизации.os

@@ -396,14 +399,19 @@

 	ИмяФайлаЛогаКоммита = ВременныеФайлы.СоздатьФайл("log");

-	авторДляГит = Автор;
-	Если Найти(Автор, "<") <= Найти(Автор, ">") Тогда
-		авторДляГит = Автор+" <"+Автор+"@localhost>"; // e-mail может быть удобен для поиска в связанных системах //авторДляГит = Автор+" <"+Автор+">";


Зачем удален код по установке базового формата имени автора для гит?
это код нужен для того, если в файле авторов указан неверный формат данных или вообще не задан файл авторов.

Там еще и AUTHORS в json превращен, насколько я понял.

EvilBeaver · 2016-11-22T13:24:47Z

Я против вливания данного PR. Часть возможностей хорошая и правильная, часть весьма спорная. Одним мегакуском это разруливать сложно. Тут аж 6 функциональных возможностей предлагается. Это, в моем понимании, должно быть 6 реквестов.

artbear · 2016-11-22T13:27:56Z

Да, я также склоняюсь к мнению, что данный PR заливать рановато.
Не ожидал, что у автора так много изменений :)

artbear · 2016-11-22T13:36:08Z

@khorevaa подскажи, для чего эта доработка?

@EvilBeaver Наше обсуждение в гиттере https://gitter.im/EvilBeaver/oscript-library?at=582ee36699dce7860aff3244 (смотреть чуть выше и ниже по тексту)

EvilBeaver · 2016-11-22T14:39:12Z

@artbear в гиттер идет обсуждение пушей через n коммитов. Отличная доработка, никто не против.
А вот переделка авторов в JSON не знаю зачем. Может и полезно, но я не вижу обсуждения с коллективом и не понимаю зачем. И насчет веток тоже обсуждения не вижу. Без явного checkout функционал пуша в заданную ветку мне непонятен.

khorevaa · 2016-11-22T14:51:14Z

Буду дробить функциональность на несколько PR...

Develop

Полный рефакторинг приложения и библиотеки

khorevaa and others added 30 commits November 15, 2016 11:51

Добавление отправки на удаленный репо каждые n - коммитов

7fbcb44

Добавлена возможность указания ветки при пакетной синхронизации

d5bc168

Исправлен вызов процедуры синхронизации с новыми параметрами

1a0a08b

Merge remote-tracking branch 'origin/master' into feature/добавление-…

e9aca5e

…параметра-ветки-в-gitsync-all # Conflicts: # src/multi-controller.os # src/xml-config.os

Добавлена возможность указания Temp каталога

bc4631d

Дорабавлен ключ "-tempcatalog"

065f230

Исправлена ошибка преобразования типов

7d55345

Automatic merge from master -> develop

001d307

* commit 'b2e6edb60af6762b933d33ac92ef5e3150a181c9':

Переработка хранения файла AUTHORS

368a923

Merge branch 'master' into new-author-format

d8cc5f0

Merge branch 'develop' into new-author-format

0522705

# Conflicts: # src/core/Классы/МенеджерСинхронизации.os

Исправлены баги.

f6218ff

Добавлены консольные команды

Очередная правка багов

706464c

Automatic merge from master -> develop

edd5938

* commit '9af89bfb6f043581cc6cd2ecc6a3d8ef50df32a3':

Доработка передачи Tags

5903a43

Доработка исправление багов

8e913cf

Переименование название параметра tempcatalog

274e370

в tempdir

Доработан автоматичесое присвоение Tags и их Push

e2b2f26

Завершено добавление-установки-тегов-по-версиям

572a829

Получение последних изменений из 'develop' into new-author-format

2e6d894

Изменен формат хранения AURHORS

3145631

Merge remote-tracking branch 'origin/new-author-format' into develop

92f3e8c

Изменен вызов команды push. Исправлены мелкие баги

7556d0a

Finish Изменение-команды-push

f31d149

nixel2007 reviewed Nov 22, 2016

View reviewed changes

EvilBeaver reviewed Nov 22, 2016

View reviewed changes

artbear requested changes Nov 22, 2016

View reviewed changes

EvilBeaver reviewed Nov 22, 2016

View reviewed changes

artbear requested changes Nov 22, 2016

View reviewed changes

khorevaa closed this Nov 22, 2016

EvilBeaver pushed a commit that referenced this pull request Feb 22, 2017

Merge pull request #6 from oscript-library/develop

88ac7ce

Develop

khorevaa deleted the Feature/Доработки-gitsync-tags-push-tempdir-authors branch May 27, 2017 12:31

artbear modified the milestone: 2.2 Oct 10, 2017

nixel2007 pushed a commit that referenced this pull request May 29, 2018

Merge pull request #6 from khorevaa/use-lib-cli

fe3acab

Полный рефакторинг приложения и библиотеки

		@@ -0,0 +1,3 @@
		// Поместите параметры в этот файл, чтобы перезаписать параметры по умолчанию и пользовательские параметры.

		КонецЕсли;


		//ПараметрыКоманды.Добавить("--all -v");

Доработки gitsync tags push tempdir authors branch #6

Доработки gitsync tags push tempdir authors branch #6

Uh oh!

Conversation

khorevaa commented Nov 22, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

khorevaa Nov 22, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

khorevaa Nov 22, 2016 •

edited

Loading