oscript-library / oscript-library/gitsync
API команды sync
Nobody has claimed this yet.
- Dominant language
- 1C Enterprise
- Stars
- 333
- Forks
- 98
- PR merge metrics
- No merged PRs in 30d
Description
artbear:
Посмотрел код sync, увидел проблему.
Значит, это API sync плохое.
Вообще, зачем кодsyncвызывать черезМенеджерКомандПриложения.ВыполнитьКоманду("sync", ЗначенияПараметров);?
Лучше выделить отдельный класс/метод, которому можно будет передать чистые параметры, без лишних/странных преобразований с магическими именами ключей
Вопрос в работе через класс "ПакетнаяСинхронизация", там
ЗначенияПараметров = Новый Соответствие;
ЗначенияПараметров.Вставить("ПутьКХранилищу", Репо.КаталогХранилища1С);
ЗначенияПараметров.Вставить("URLРепозитория", Репо.GitURL);
ЗначенияПараметров.Вставить("ЛокальныйКаталогГит", Репо.КаталогВыгрузки);
ЗначенияПараметров.Вставить("-email", Репо.ДоменПочтыДляGit);
ЗначенияПараметров.Вставить("-v8version", Репо.ПутьКПлатформе83);
ЗначенияПараметров.Вставить("-push-every-n-commits", Репо.КоличествоКоммитовДоPush);
ЗначенияПараметров.Вставить("-check-authors", ПреобразоватьЗначениеКБулево(Репо.ПроверитьАвторовХранилища));
ЗначенияПараметров.Вставить("-stop-if-empty-comment", ПреобразоватьЗначениеКБулево(Репо.ПрерватьВыполнениеБезКомментарияКВерсии));
ЗначенияПараметров.Вставить("-auto-set-tags", ПреобразоватьЗначениеКБулево(Репо.АвтоматическаяУстановкаТэговПоВерсиям));
ЗначенияПараметров.Вставить("-process-fatform-modules", ПреобразоватьЗначениеКБулево(Репо.ПереименовыватьФайлМодуляОбычнойФормы));
// дополним настройками из параметров команды
Для Каждого ПараметрКоманды Из мПараметрыКоманды Цикл
ЗначенияПараметров.Вставить(ПараметрКоманды.Ключ, ПараметрКоманды.Значение);
КонецЦикла;
МенеджерКомандПриложения.ВыполнитьКоманду("sync", ЗначенияПараметров);
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the ПакетнаяСинхронизация code and the МенеджерКомандПриложения.ВыполнитьКоманду entry point shown in the issue. Trace how sync parameters are assembled and transformed, then define the affected call sites and tests before changing the API. Done means the proposed sync interface and parameter handling are agreed and consistently used.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- git
- Domain
- cli, tooling
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100