-
Notifications
You must be signed in to change notification settings - Fork 93
Feature/fix 88 #88 #93
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
Conversation
Определяем мин количество коммитов между разницей версии уже в git и в хранилище для случаев, когда пересоздали хранилище и в гите находится версия 1234, а в хранилище 4, тогда вызываем исключение если разница в коммитах больше чем на 10 версий. Т.к. синхронизация пойдет дальше только тогда, когда сами заметят или же версия в новом хранилище будет больше чем 1234
artbear
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Идея интересная, но при использовании такой реализации будут проблемы:
- нет возможности управления этой фичей
- при обрезании хранилища существующий репозиторий Гит вообще невозможно будет синхронизировать, т.к. гитсинк будет постоянно выдавать ошибку
- и без изменения кода гитсинка ничего сделать нельзя
Предлагаю
- или добавить очередной ключ :(
- или переработать идею
а зачем управление, если это страховка и предупреждение. Какие могут быть ситуации, когда у нас версия в файле VERSIONS - является больше чем версия в хранилище и это правильно? Именно больше и не равно?
при обрезании хранилища, вообще ничего не будет происходить, тупо возвращаться с кодом возврата 0. Управление тут простое - ошибка, или же меняем вручную файл Versions, причем заметь, даже в текущем варианте, нам необходимо менять файл вручную, если обрезали хранилище. Имхо, как страховка идея и реализация норм, единиственное, что добавил бы это более информативное сообщение типа Лог.Предупрждение("Возможно обрезали хранилище, измените файл Versions и исправьте %1 на правильную версию равную <= версии в хранилище %2, ВерсияВерсион, ВерсияХранилища); |
|
С таким пояснением согласен. Добавишь предупреждение? Предлагаю |
|
Исправил, но только не Предупреждении, а Ошибку вывожу, все равно же будет дальше исключение показываться. |
Определяем мин количество коммитов между разницей версии уже в git и в хранилище для случаев, когда пересоздали хранилище и в гите находится версия 1234, а в хранилище 4, тогда вызываем исключение если разница в коммитах больше чем на 10 версий.
Т.к. синхронизация пойдет дальше только тогда, когда сами заметят или же версия в новом хранилище будет больше чем 1234