Feature/custombinpath #79
Conversation
Исправленна документация. Добавленно чтение настроек opm.cfg с различных путей системных настроек и настроек пользователя.
README.md
Outdated
| opm update -all | ||
| ``` | ||
|
|
||
| ## Настройка прокси серевера для скачивания пакетов |
There was a problem hiding this comment.
Опечатка - правильно "прокси-сервера"
artbear
left a comment
artbear
left a comment
There was a problem hiding this comment.
Хороший PR
но нужны небольшие дополнения
src/Классы/УстановкаПакета.os
Outdated
There was a problem hiding this comment.
Как-то некрасиво смотрится использование англоязычной функции в русском коде :(
Предлагаю исправить на русское имя
src/Модули/НастройкиПриложенияOpm.os
Outdated
There was a problem hiding this comment.
Опечатка - правильно НастройкиНайдены (одна н)
src/Модули/НастройкиПриложенияOpm.os
Outdated
There was a problem hiding this comment.
Идет дублирование магической строки "opm.cfg", хотя уже есть спец.публичная переменная ИмяФайлаНастроек в КонстантыOpm.os
There was a problem hiding this comment.
Магия Модулей и Классов и их использование одного из другого - пишет ошибку при использовании из константы.
src/Модули/НастройкиПриложенияOpm.os
Outdated
src/Модули/НастройкиПриложенияOpm.os
Outdated
There was a problem hiding this comment.
Вместо использования НастройкиНайдены предлагаю юзать Прервать сразу после успешного чтения файла настроек, код становится проще и понятнее.
src/Модули/НастройкиПриложенияOpm.os
Outdated
There was a problem hiding this comment.
Здесь нужно поменять порядок согласно поведению и доке
Правильно
текущий каталог запуска, домашний каталог пользователя, системная настройка, Каталог opm (для совместимости)
README.md
Outdated
| ``` | ||
|
|
||
| ## Настройка прокси серевера для скачивания пакетов | ||
| ## Настройка прокси-серевера для скачивания пакетов |
src/Модули/НастройкиПриложенияOpm.os
Outdated
There was a problem hiding this comment.
Используй Если Не НастройкиНайден Тогда - и Сонар, и стандарт 1С ругается
|
В мастер :( |
|
Женя, косяк :( |
3 participants
Добавил чтение переменной окружения для настройки пути создания скриптов. OSCRIPTBIN - можно переопределять пути для создания bat/sh файлов.
Исправленна документация.
Добавленно чтение настроек opm.cfg с различных путей системных настроек и настроек пользователя.