Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

precommit.config #85

Open
zeratulayuris opened this issue May 12, 2016 · 14 comments
Open

precommit.config #85

zeratulayuris opened this issue May 12, 2016 · 14 comments
Milestone

Comments

@zeratulayuris
Copy link

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

@nixel2007
Copy link
Member

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

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

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

@zeratulayuris
Copy link
Author

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

@nixel2007
Copy link
Member

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

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

@zeratulayuris
Copy link
Author

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

@zeratulayuris
Copy link
Author

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

@nixel2007
Copy link
Member

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

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

@ghost
Copy link

ghost commented May 15, 2016

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

@zeratulayuris
Copy link
Author

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

@zeratulayuris
Copy link
Author

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

@zeratulayuris
Copy link
Author

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

@zeratulayuris zeratulayuris added this to the 2.2.2 milestone Mar 24, 2019
@artbear
Copy link
Member

artbear commented Apr 10, 2019

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

@artbear
Copy link
Member

artbear commented Apr 10, 2019

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

@zeratulayuris
Copy link
Author

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

@zeratulayuris zeratulayuris modified the milestones: 2.2.2, next Apr 11, 2019
@zeratulayuris
Copy link
Author

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

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

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

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

No branches or pull requests

3 participants