precommit1c icon indicating copy to clipboard operation
precommit1c copied to clipboard

precommit.config

Open zeratulayuris opened this issue 8 years ago • 14 comments

В связи с #68 и #84 родилась мысль читать часть параметров из файла конфигурации и поставлять precommit.config.sample, чтобы дать представление о возможных параметрах. Ваше мнение?

zeratulayuris avatar May 12 '16 17:05 zeratulayuris

Такое даже было раньше, в python-версии. Только нужно сначала обсудить список параметров.

Точно нужен путь к платформе, уже поступало несколько запросов по этому поводу.

Путь к oscript в целом можно переопределить в самом файле pre-commit. Что еще может понадобиться?

nixel2007 avatar May 12 '16 17:05 nixel2007

Нужен флаг использования конфигуратора, если (когда :)) #84 смержится. Нужны параметры подключения к ИБ. Можно подумать насчет версии платформы.

zeratulayuris avatar May 13 '16 09:05 zeratulayuris

Нужен флаг использования конфигуратора, если (когда :)) #84 смержится. Нужны параметры подключения к ИБ.

так это ж все в самом файле pre-commit указывается в флагах.

nixel2007 avatar May 13 '16 10:05 nixel2007

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

zeratulayuris avatar May 13 '16 11:05 zeratulayuris

Тем более, что я читал про планы сделать precommit глобальным - тогда и вовсе сам хук лучше не править, если он будет добавляться в репо, как симлинк.

zeratulayuris avatar May 13 '16 11:05 zeratulayuris

Тем более, что я читал про планы сделать precommit глобальным - тогда и вовсе сам хук лучше не править, если он будет добавляться в репо, как симлинк.

ммм... изначально просто из консоли будет доступна команда precommit install, которая будет копировать файлы в каталог с хуками. На сколько я помню, идей с симлинками никто не озвучивал.

nixel2007 avatar May 13 '16 11:05 nixel2007

небольшой офтопик @zeratulayuris - я смотрю у тебя в форке много коммитов и изменений, это по какой-то задаче ? конфликтов не будет ? просто на следующей неделе начинается веселье с параметрами сборки для behavior, могут быть изменения.

ghost avatar May 15 '16 08:05 ghost

@allustin пока вроде без конфликтов, всё можно увидеть в #84 - это к #68 относится и чуть-чуть уборки.

zeratulayuris avatar May 16 '16 06:05 zeratulayuris

@nixel2007 думаю, стоит озвучить идею с симлинками - тогда можно будет обновлять хуки просто сделав pull в одном репозитории. Но это только как вариант, кому-то наверняка подойдёт выборочное обновление.

zeratulayuris avatar May 16 '16 06:05 zeratulayuris

Возвращаясь к теме с файлом конфига. Есть ли что-то, кроме #114, требующее этого файла?

zeratulayuris avatar Mar 24 '19 13:03 zeratulayuris

Имя каталога src также лучше бы в конфиг положить.

artbear avatar Apr 10 '19 10:04 artbear

ну и свежая багофича - хранение текстового представления макета рядом с мхл-файлом - также требует настройки

artbear avatar Apr 10 '19 10:04 artbear

Ещё корень бинарников. Т.е. вести отсчет относительных путей к бинарным файлам не от корня репозитория, а от bin, например.

zeratulayuris avatar Apr 10 '19 11:04 zeratulayuris

Однако появились мысли:

  • файл конфига по-умолчанию назвать .precommit1c и хранить в корне репо - так проще переносить конфиг между репо разработчиков
  • добавить аргумент командной строки --config-file, позволяющий использовать произвольный файл конфига
  • в конфиге позволить переопределять все опции, которые доступны через командную строку, кроме --config-file

Соответственно при выполнении считываются параметры из файла .precommit1c в каталоге запуска, потом из файла --config-file, потом применяются параметры из аргументов запуска.

zeratulayuris avatar Nov 09 '19 18:11 zeratulayuris