Skip to content

Conversation

@nixel2007
Copy link
Member

По итогам oscript-library/gitsync#12

Механизм проверки версии взят из nixel2007/mtgcm (а там он был взят из бсп и переведен на английский) и временно включен в сам файл установщика. Сейчас хочу показать вам концепцию. Если все устроит, то перенесу в пакет strings

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ничего не сказано про случай, когда первая версия меньше второй.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это перевод официальной документации с БСП :Р Там тоже нет.

Но добавить имеет смысл, согласен.

@artbear
Copy link
Member

artbear commented Dec 2, 2016

Больше возражений нет.
Концепция устраивает

@nixel2007
Copy link
Member Author

@EvilBeaver мне подсказал, что уже есть функция проверки версий.
@artbear убрал вызов, документировать больше не нужно ;)
Заодно поправил обнаруженный баг.

@artbear
Copy link
Member

artbear commented Dec 2, 2016

ИМХО можно мержить.

@nixel2007 nixel2007 merged commit 5d404aa into master Dec 2, 2016
@nixel2007 nixel2007 deleted the feature/check-oscript-version branch December 2, 2016 14:45
@nixel2007 nixel2007 restored the feature/check-oscript-version branch December 3, 2016 22:05
@nixel2007 nixel2007 deleted the feature/check-oscript-version branch December 3, 2016 22:06
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