ОСенний OVM #20
ОСенний OVM #20
Conversation
| @@ -38,11 +49,11 @@ | |||
| // КомандаПриложения - КомандаПриложения - Выполняемая команда | |||
| // | |||
| Процедура ВыполнитьКоманду(Знач КомандаПриложения) Экспорт | |||
|
|
|||
| МассивВерсийКУстановке = КомандаПриложения.ЗначениеАргумента("VERSION"); | |||
There was a problem hiding this comment.
А тут специально не стал переводить на осень опции?
There was a problem hiding this comment.
Там опциональная опция есть, типа добавляется по условию, autumn-cli так не умеет
There was a problem hiding this comment.
Там опциональная опция есть, типа добавляется по условию, autumn-cli так не умеет
Заведешь ишусь пожалуйста? 🙂
There was a problem hiding this comment.
Посмотрел код команды, чо-т вообще не особо себе представляю, как это сделать на уровне метаданных. Хотя можно попробовать указать имя функции-предиката. Ну, либо выпилить условие :)
There was a problem hiding this comment.
Посмотрел код команды, чо-т вообще не особо себе представляю, как это сделать на уровне метаданных. Хотя можно попробовать указать имя функции-предиката. Ну, либо выпилить условие :)
Можно в теории сделать аннотацию которая на входе будет принимать лямбду и захватывать объект в котором аннотация расположена.
There was a problem hiding this comment.
Там опциональная опция есть, типа добавляется по условию, autumn-cli так не умеет
Заведешь ишусь пожалуйста? 🙂
|
Ждём релиза новой версии Ну а так упакованный в exe работает пуще прежнего
|
@nixel2007 можешь дать Кириллу права на кли и логос осенние, чтоб он пушнул в хаб? Я черт знает когда доберусь до компа |
|
Ща разберёмся |
| Новый Структура("УстановленнаяВерсия", УстановленнаяВерсия) | ||
| ) | ||
| .Первые(1) | ||
| .Обработать("(Элемент) -> |
There was a problem hiding this comment.
Версию флюента не надо бампнуть в пэкэдждеф?
There was a problem hiding this comment.
Ага, надо, бампанем
| Новый Структура("УстановленнаяВерсия", УстановленнаяВерсия)) | ||
| .Первые(1) | ||
| .Обработать(" | ||
| |Если ВерсииOneScript.ЭтоКаталогВерсии2(Элемент.Путь) Тогда |
There was a problem hiding this comment.
Просто космос, конечно. Восхитительно
| КонецФункции | ||
|
|
||
| Функция ВерсияПриложения() Экспорт | ||
| Возврат "1.1.0"; |
src/core/Классы/ФайлНастроек.os
Outdated
| &Желудь | ||
| Процедура ПриСозданииОбъекта(&ЛогOVM пЛог) | ||
|
|
||
| КаталогУстановкиПоУмолчанию = ПолучитьПеременнуюСреды("OVM_INSTALL_PATH"); |
There was a problem hiding this comment.
А это сюда откуда перетекло? Ответственность не размазалась, случаем?
There was a problem hiding this comment.
Из параметров овм, но параметры овм зависят от файла настроек так что циклическое внедрение желудей получалось, развязал зависимость вот так, есть идеи получше?)
There was a problem hiding this comment.
- по идее финальный штрих был разработан как раз для решения проблемы циклической зависимости, но я не уверен, что он в итоге был так разработан.
- по идее это самая настоящая деталька. Но для поддержки переменных среды нужно в осени реализовать провайдер настроек для конфигора из переменных среды. Вроде как частично его можно выдрать из опм.
There was a problem hiding this comment.
- Не очень понимаю как мне финальный штрих поможет запластилинить параметры овм в конструктор.)
- Ну можно так же подождать пока осень доработаем, все равно ждём переопределение настроек
There was a problem hiding this comment.
- Не в конструктор. Ты всю инициализацию тоже делаешь в финальном штрихе. А пластилин - в поля (по крайней мере в одном классе, но может и в обоих)
There was a problem hiding this comment.
Но глобально, циклическая зависимость - это плохо, да. Думаю подход 2) это как раз развязывание инфры и логики. Может провайдер настроек ещё и для ini сделать, тогда все можно будет инжектить детальками напрямую.
There was a problem hiding this comment.
Провайдер .ini ещё в конфигор? Типа нам нужен будет провайдер cli провайдер env и провайдер ini
Co-authored-by: Nikita Fedkin <[email protected]>
…приложения значением деталек 2. Вся работа с чтением настроек из файла настроек отдана ОСени, ovm теперь только записывает настройки в .ini файл 3. Сам файл настроек переименован ovm.config -> autumn-properties.ini после обновления ovm если были заданы настройки нужно будет переименовать существующий файл в новое имя для корректной работы
Co-authored-by: Nikita Fedkin <[email protected]>
| .ЗависитОт("strings") | ||
| .ЗависитОт("decorator", "1.5.2") |
There was a problem hiding this comment.
А он зачем? Не вижу импортов
There was a problem hiding this comment.
Импорты у autumn-cli/autumn-logos но там импорты версии которая не поддерживает упаковку в exe, овм просто задирает версию чтоб собраться
There was a problem hiding this comment.
А, ну так давай там бампнем сразу, чего тут тащить.
Перевод проекта на рельсы осени
Closes #18