search

bia-technologies / precommit4onec

Утилита для автоматической обработки исходных файлов конфигурации, внешних отчетов и обработок для платформы 1С:Предприятие при помещении в репозиторий git
MIT License
54 stars 18 forks source link

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

Open artkor83 opened 2 years ago

realMaxA commented 2 years ago

Как оперативно. Только что PR сделал на использование параметров подключения к ИБ в прекоммите (https://github.com/bia-technologies/precommit4onec/pull/18). Правда я добавил ещё эти параметры в install. Можно объединить эти PR.

artkor83 commented 2 years ago

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

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

realMaxA commented 2 years ago

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

artkor83 commented 2 years ago

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

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

realMaxA commented 2 years ago

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

theshadowco commented 2 years ago

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

IT-Medved commented 1 year ago

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

theshadowco commented 1 year ago

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

IT-Medved commented 1 year ago

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