Skip to content

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

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

Merged
merged 1 commit into from
Jul 4, 2022

Conversation

infina15
Copy link
Contributor

@infina15 infina15 commented Jul 1, 2022

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

@EvilBeaver
Copy link
Member

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

@infina15
Copy link
Contributor Author

infina15 commented Jul 1, 2022

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

@EvilBeaver
Copy link
Member

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

@infina15
Copy link
Contributor Author

infina15 commented Jul 1, 2022

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

Сравнил версию v8storage, которая сейчас используется в gitsync (0.9.3) и последнюю (0.9.4): oscript-library/v8storage@0.9.3...0.9.4

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

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

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

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

@EvilBeaver EvilBeaver merged commit 287f1f1 into oscript-library:develop Jul 4, 2022
@infina15
Copy link
Contributor Author

infina15 commented Jul 4, 2022

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

@EvilBeaver
Copy link
Member

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

@nixel2007
Copy link
Member

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

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

Successfully merging this pull request may close these issues.

3 participants