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

Добавить возможность блокировать конвертацию версии хранилища с пустым комментарием #147

Open
mrshadow300373 opened this issue Nov 30, 2020 · 3 comments

Comments

@mrshadow300373
Copy link

Комментарий принимается один единственный раз, при создании отчёта по версиям, и если его нет, то фиксация влетает в гит с комментарием <Комментарий не указан>, и потом уже даже если разработчик спохватился и решил написать, то уже поздно. Переписывать коммиты в гит потом - плохая идея, в особенности если изменения из этой ветки куда то слили. Отсутствие же комментариев усложняет поиск безобразников в коде сводя к минимуму возможности git-blame.

Нужно добавить опцию, останавливающую конвертацию. при которой бы, если комментария к версии нет, не загружать её и все последующие пока он там не появится.

Далее возможны любые сценарии, например в почту зарегистрированную прислать гневное письмо, об отсутствующем комментарии, либо ничего не делать просто когда выясниться что в гит изменения не прилетели админ попинает разработчика, видя, что его хранилище 1С нет комментариев.

@marmyshev
Copy link
Collaborator

Была сделана доработка как раз чтобы забивать пустой комментарий текстом <Комментарий не указан>.

Но вот останавливать ли конвертацию, когда не указан - это большой вопрос. Скорее всего это понадобится далеко не всем.
Я с самой проблемой согласен - что если не указали коммент - это плохо. и плохо потом переписывать историю Git.

Вопрос как дальше обновлять пустые комменты? наверное надо заново запрашивать отчет по версиям для всех пустых. Если у вас многотысячная история версий в хранилище - заполнять наверное нужно сначала все перед стартом конвертации.

а дальше - уже конроллировать в таком жестоком режиме...

Может быть это сделать в виде расширения? Вполне годная функциональность для расширения.

@mrshadow300373
Copy link
Author

mrshadow300373 commented Nov 30, 2020

Вопрос как дальше обновлять пустые комменты?
Да никак, пока комментарий в хранилище 1С не поставит, не создавать новую версию в справочнике версий, и потом хоть десяток пусть накидает. Если добавит комментарий к той фиксации, которую пропустил, то создать и загрузить эту версию и все последующие, пока не упрёмся в очередной, в котором опять нет комментария, насколько я понял при конвертации каждое помещение в хранилище 1С всё равно обособленно обрабатывается.
Скорее всего это понадобится далеко не всем.
Согласен, но вот мне не критично подождать пусть лучше не загрузится в гит конвертере, чем потом улетит в гит. Не долетит до гита, не будет обновления ветки. не будет сборки тестовой базы, в итоге не будет денег.
Может быть это сделать в виде расширения?
Просто добавить расширение в новый каталог внутри репо и зареквестить? Или ещё каким то образом?

@marmyshev
Copy link
Collaborator

Просто добавить расширение в новый каталог внутри репо и зареквестить? Или ещё каким то образом?

Да, посмотрите как сделаны остальные расширения. И правила контрибутинга, лицензия, создавайте реквест...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants