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

Доработать код работы с хранилищем из-за изменений в v8storage #193

Closed
artbear opened this issue Nov 30, 2017 · 6 comments
Assignees
Milestone

Comments

@artbear
Copy link
Collaborator

artbear commented Nov 30, 2017

https://github.com/Khorevaa/v8storage/releases/tag/0.6.4
Переименованы функции, которые используются в раннере

@khorevaa ты меняешь свое API и ломаешь совместимость, а отмечаешь изменения только в patch-номере :(
нехорошо :(

ладно, хоть я зашел и почитал изменения в релизе.
иначе был бы сюрприз для пользователей ванесса-раннер!
я бы выпустил версию 0.7.0 для этого

@artbear artbear added this to the 0.18.0 milestone Nov 30, 2017
@artbear
Copy link
Collaborator Author

artbear commented Jan 25, 2018

CI сейчас падает именно из-за этого
http://ci.silverbulleters.org/job/Vanessa-runner/job/develop/lastCompletedBuild/consoleFull

ИНФОРМАЦИЯ - Выполняю создание пользователя хранилища конфигурации
17:04:50 КРИТИЧНАЯОШИБКА - 
{Модуль C:\cicd\workspace\Vanessa-runner_develop
QMF2XLOCAPVNBGZY6TVHLNWTPUR6NAPQUNBIN3JDPZYKWTF2Y64Q\
src\Классы\КомандаСоздатьПользователейХранилища.os / 
Ошибка в строке: 93 / {
Модуль C:\cicd\workspace\Vanessa-runner_develop
QMF2XLOCAPVNBGZY6TVHLNWTPUR6NAPQUNBIN3JDPZYKWTF2Y64Q\
src\Классы\МенеджерКонфигуратора.os / 
Ошибка в строке: 336 / Метод объекта не обнаружен (УстановитьКаталогХранилища)}    
17:04:50 	ХранилищеКонфигурации.УстановитьКаталогХранилища(ПутьХранилища);

@nixel2007
Copy link
Contributor

0.x.x - имеет право делать все, что хочет.
Это твоя обязанность как потребителя 0.x библиотеки следить за ее API

@artbear
Copy link
Collaborator Author

artbear commented Jan 25, 2018

про имеет право не совсем согласен.
ломание совместимости все-таки достойно более серьезной правки версии.

@nixel2007
Copy link
Contributor

Major version zero (0.y.z) is for initial development. Anything may change at any time. The public API should not be considered stable.

@artbear
Copy link
Collaborator Author

artbear commented Jan 25, 2018

Исправлено

@artbear artbear closed this as completed Jan 25, 2018
@artbear artbear self-assigned this Jan 25, 2018
@khorevaa
Copy link

khorevaa commented Jan 26, 2018

@artbear @nixel2007 Следующий раз учту.

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

3 participants