search

oscript-library / gitsync

Синхронизация хранилища 1С с репозиторием git
Mozilla Public License 2.0
269 stars 91 forks source link

В хранилище метка и комментарий метки добавлены 2 года назад, пора см… #292

Closed infina15 closed 2 years ago

infina15 commented 2 years ago

В v8storage давно добавили поля "Метка" и "КомментарийВерсии" для отчета хранилища. Предлагаю обновить зависимость в gitsync.

EvilBeaver commented 2 years ago

Я не возражаю. Пришлёте пулреквест?

infina15 commented 2 years ago

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

EvilBeaver commented 2 years ago

Все сделано правильно, но у меня вопрос - достаточно ли только поменять версию? Разве не должно быть еще что-то внутри гитсинка дописано, чтобы новые поля "Метка" и "КомментарийВерсии" начали использоваться?

infina15 commented 2 years ago

Все сделано правильно, но у меня вопрос - достаточно ли только поменять версию? Разве не должно быть еще что-то внутри гитсинка дописано, чтобы новые поля "Метка" и "КомментарийВерсии" начали использоваться?

Сравнил версию v8storage, которая сейчас используется в gitsync (0.9.3) и последнюю (0.9.4): https://github.com/khorevaa/v8storage/compare/0.9.3...0.9.4

Там изменены только тесты и вот эти комментарии. Дело в том, что в текущем состоянии зависимостей v8storage получает таблицу изменений хранилища при заполненной метке вот так: изображение

Потому что видит что начался комментарий, потом читает и сам комментарий, потом метку, ее значение, потом комментарий метки и ее значение. В моем примере комментарий заполнен, а комментарий метки нет, поэтому в сухом остатке у меня в комментарии коммита остается "Комментарий метки:". Вот например результат как ищет регулярка в v8storage: изображение

Так работает на 8.3.20.

Предлагаю пока обновить только версию зависимости, чтобы гитсинк мог ставить корректно хотя бы сам комментарий. Далее можно будет думать как отображать комментарий, метку и комментарий метки в одном поле коммита в гите. Кстати, есть идеи как лучше?

infina15 commented 2 years ago

@EvilBeaver а когда теперь это изменение можно будет получить через opm update?

EvilBeaver commented 2 years ago

@khorevaa @nixel2007 @artbear кто у нас теперь выпускает релизы gitsync?

nixel2007 commented 2 years ago

@EvilBeaver GA: https://github.com/oscript-library/gitsync/actions/runs/2610145947