Интерактивная обработка внешних файлов (без коммит-хука)

[artkor83]

No description provided.

[realMaxA]

Как оперативно. Только что PR сделал на использование параметров подключения к ИБ в прекоммите (#18). Правда я добавил ещё эти параметры в install.
Можно объединить эти PR.

[artkor83]

Как оперативно. Только что PR сделал на использование параметров подключения к ИБ в прекоммите (#18). Правда я добавил ещё эти параметры в install. Можно объединить эти PR.

Ради бога, если сможете сделайте (я просто не делал никогда мерджи2х PR). Доработки там небольшие: интерактивный запуск без коммит-хука: я обычно в корне репо делаю батничек с командой прекоммита.

[realMaxA]

Думаю мерджи тут не нужны. Тут варианта два: или ты добавляешь в команду install указание -ibconnection, -u и -p, или я добавляю интерактивный запуск. То на то и выйдет -- посмотри, как я изменил install, если не нравится, перепиши. Также у нас по разному названы параметры в справке, тоже можно объединить и взять лучшее.
И ещё один момент: я влил в develop изменения из master, уж не знаю, почему Валерий взялся коммитить сразу в мастер. Здесь тоже надо понять, берём мы в develop эти изменения или нет. Думаю без автора здесь не обойтись. Пусть рассудит.

[artkor83]

Думаю мерджи тут не нужны. Тут варианта два: или ты добавляешь в команду install указание -ibconnection, -u и -p, или я добавляю интерактивный запуск. То на то и выйдет -- посмотри, как я изменил install, если не нравится, перепиши. Также у нас по разному названы параметры в справке, тоже можно объединить и взять лучшее. И ещё один момент: я влил в develop изменения из master, уж не знаю, почему Валерий взялся коммитить сразу в мастер. Здесь тоже надо понять, берём мы в develop эти изменения или нет. Думаю без автора здесь не обойтись. Пусть рассудит.

Немного запутался: кинь в меня ссылкой на свой актуальный PR с последним коммитом. Наверное туда добавлю свои фрагменты кода (их немного)

[realMaxA]

Вот этот PR (рядом лежит -- https://github.com/bia-technologies/precommit4onec/pulls): #18.
Вот изменения для install: d463a82

[theshadowco]

А зачем это? Есть же exec-rules?

[IT-Medved]

@theshadowco Идея хорошая кстати, exec-rules'у не важно реп это или просто директория, он все обрабатывает.
А тут более точнечная настройка прекоммит команды. Если вешать ее на кнопку в том же ГЕ то каждый получит что ему надо. Напр, если повесить на кнопку, мне было бы удобнее обрабатывать то что в индексе чтобы глянуть дифф т.е. стандартное поведение. А кому-то, наоборот, как в этом реквесте удобнее было бы обработать все

[theshadowco]

Интересный кейс, принято, надо будет ребейзнуть ток

[IT-Medved]

@artkor83, Привет, отребейзите?