Skip to content

Conversation

@Borisskin
Copy link
Contributor

В процедуру добавлен параметр указания количества заданий загрузки из dt, в коде есть проверка, что этот параметр используется только с платформы версии 8.319.
По-умолчанию, параметр = 0, как и указано в ИТС, если параметр не указан.

@artbear artbear added this to the v1.9.0 milestone Jun 15, 2022
@artbear artbear merged commit 04f6d64 into oscript-library:develop Jun 15, 2022
@artbear
Copy link
Member

artbear commented Jun 15, 2022

@Borisskin Большое спасибо! Поздравляю с первым ПР в проекте!

@artbear
Copy link
Member

artbear commented Jun 15, 2022

@Borisskin в Ванесса-раннер сделаешь подобную доработку?

@Borisskin Borisskin deleted the feature/JobsCount branch June 15, 2022 18:49
@Borisskin
Copy link
Contributor Author

@Borisskin в Ванесса-раннер сделаешь подобную доработку?

Доработка у меня в ветке готова, но не выложено. Я помню, что ты говорил "отходим от именованных параметров к позиционным". Но как указать позиционный, который опциональный? У меня пока такое предложение
Парсер.ДобавитьИменованныйПараметрКоманды(ОписаниеКоманды, "--JobsCount ", "Количество заданий для загрузки информационной базы. Значение по умолчанию ‑ 0 и равно числу ядер");

artbear added a commit that referenced this pull request Jun 19, 2022
…и т.д. (#144)

* Выравниваю master и develop, вливая master (#138)

* накидал функцию

* вынес допустимые ключи в отдельную функцию

Co-authored-by: Nikita Gryzlov <[email protected]>
Co-authored-by: Голованов Сергей <[email protected]>
Co-authored-by: Andrei Ovsiankin <[email protected]>

* Добавлена логика формирования строки соединения

Сделано для упрощения понимания

* Доработан алгоритм формирования строки соединения из параметров

Дополнен файл описания
Оптимизированы алгоритмы формирования строки
Поправлены тесты под новую логику

* Исправлены опечатки

* oscript_modules в гитигнор

* Добавление подключения к хранилищу

* Параметр количества заданий загрузки из dt (#142)

* Параметр количества заданий загрузки dt

* Для JobsCount платформа с 8.3.19

* Опечатка добавленного параметра

Co-authored-by: Nikita Gryzlov <[email protected]>
Co-authored-by: Голованов Сергей <[email protected]>
Co-authored-by: Andrei Ovsiankin <[email protected]>
Co-authored-by: Vdovenko Sergey <[email protected]>
Co-authored-by: 8095tores <[email protected]>
Co-authored-by: Boris Sinitsyn <[email protected]>
artbear added a commit that referenced this pull request Jun 29, 2024
* Выравниваю master и develop, вливая master (#138)

* накидал функцию

* вынес допустимые ключи в отдельную функцию

Co-authored-by: Nikita Gryzlov <[email protected]>
Co-authored-by: Голованов Сергей <[email protected]>
Co-authored-by: Andrei Ovsiankin <[email protected]>

* Добавлена логика формирования строки соединения

Сделано для упрощения понимания

* Доработан алгоритм формирования строки соединения из параметров

Дополнен файл описания
Оптимизированы алгоритмы формирования строки
Поправлены тесты под новую логику

* Исправлены опечатки

* oscript_modules в гитигнор

* Добавление подключения к хранилищу

* Параметр количества заданий загрузки из dt (#142)

* Параметр количества заданий загрузки dt

* Для JobsCount платформа с 8.3.19

* Опечатка добавленного параметра

* увеличил номер версии

* Отчет сравнения конфигураций по двум файлам

* Исправление описания параметра

* Опечатка - задвоение команды добавления параметра

* Опечатка в названии параметра

* Рефакторинг параметров запуска

* Инкремент версии

* Реализована выгрузка информационной базы в текущий каталог если в параметрах указано только имя файла выгрузки

* изменил алгоритм проверки пути

* Проверка заменена на формирование абсолютного пути

* Github workflow (#162)

Co-authored-by: Artur Ayukhanov <[email protected]>

* Версия среды 1.9 + последние версии зависимостей

---------

Co-authored-by: Nikita Gryzlov <[email protected]>
Co-authored-by: Голованов Сергей <[email protected]>
Co-authored-by: Andrei Ovsiankin <[email protected]>
Co-authored-by: Vdovenko Sergey <[email protected]>
Co-authored-by: 8095tores <[email protected]>
Co-authored-by: Boris Sinitsyn <[email protected]>
Co-authored-by: Alexander Medvedko <[email protected]>
Co-authored-by: Artur Ayukhanov <[email protected]>
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.

2 participants