Переход с cmdline на cli #127
Conversation
|
Конфликты |
src/cmd/Классы/КомандаOpm_Push.os
Outdated
There was a problem hiding this comment.
Я не силен в CLI. А разве опция file и аргумент FILE не мешают друг-другу?
There was a problem hiding this comment.
не должны. они лежат в разных коллекциях.
There was a problem hiding this comment.
@EvilBeaver Вообще никак не мешают друг другу! )) Работать будет )
There was a problem hiding this comment.
@EvilBeaver @nixel2007 Оцените перечисления в опциях ))
src/cmd/Классы/КомандаOpm_Push.os
Outdated
There was a problem hiding this comment.
Как работают опция и аргумент одновременно?
There was a problem hiding this comment.
Сейчас заложена логика как было
Сначала проверка на наличии указания файла потом аргумент FILE
Опцию файл оставил только для совместимости! По идее ее можно со временем убрать
There was a problem hiding this comment.
Лучше убрать сразу. Меня всегда аргумент --file раздражал. неясно заячем он
There was a problem hiding this comment.
@EvilBeaver опция? file? Убрать? Давай в следующей версии! ))) Чтобы людей не шокировать !) А то я вон с gitsync наделал - теперь народу страшно с новой строкой использования!
There was a problem hiding this comment.
@EvilBeaver Сейчас можно какое нибудь предупреждение сделать )
There was a problem hiding this comment.
От указания файла вообще лучше отказаться.
Создал ишуз #128
Но отказаться в след.релизе
src/cmd/Классы/КомандаOpm_Push.os
Outdated
|
Ни одна сборка не прошла тестирование. |
|
@artbear Тесты все проходят.. Кроме 1bdd - они раньше на ci - не выполнялись.. |
|
Если у @nixel2007 возражений нет - надо вливать |
|
@khorevaa Про 1bdd напиши чуть подробнее, что не так и кто виноват? |
вливать можно только, если пройдут ВСЕ СУЩЕСТВУЮЩИЕ в проекте тесты, иначе гарантированно нарвемся на проблемы совместимости. |
|
и узнаем о них только, когда кто-то попробует установить новый опм. |
|
@artbear а тесты на старом opm работали все-все-все? Сдается мне, что они и раньше не работали |
|
@artbear по 1bdd виноват - я! Когда делал рефакторинг в версии 0.14 тесты 1bdd через opm test не запускались я их и не чинил под новую структуру |
|
@artbear @EvilBeaver Тесты я поправлю, но только уже через 2 недели. (отпуск) Можно пока влить в девелоп или кто-то может помочь с тестами |
|
Я прогоню тесты на старом. Если и на старом падают - починю и волью. |
|
@EvilBeaver @khorevaa Напоминаю, что тесты встроенных библиотек нужно запускать не через чтобы загружались версии из разработческого каталога, а не из системного каталога. |
|
@EvilBeaver @artbear @nixel2007 Исправил тесты! Все тесты на travis-ci проходят https://travis-ci.org/khorevaa/opm/jobs/410478241 |
|
@EvilBeaver @artbear @nixel2007 Вливаем? ) |
4 participants
Переведены все команды на cli