-
Notifications
You must be signed in to change notification settings - Fork 17
Детализация ошибок при парсинге опций и аргументов #78 #79
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
base: develop
Are you sure you want to change the base?
Conversation
|
@khorevaa посмотри, пожалуйста |
|
Пока незакончено, переделать кое что нужно. Поспешил ( |
|
Меня несколько смущает добавление флага Обязательный на уровень парсера. Насколько я помню, это раньше и так хранилось в классах, описывающих опции и параметры. Как-то оно же работало раньше :) есть риск, что абстракция потекла. |
|
Вот это и переделываю) |
|
Аргументы - всегда обязательны кроме случаев когда он один. (это кейс специально сделан) То что тут написано (накодено), я уже забыл... так что от меня мало толка )) |
|
Готово к ревью |
|
@nixel2007 @khorevaa Давайте уж вольём, а? Вроде работает :) |
Добавлена детализация ошибки
Ошибка чтения параметров команды:-x-x-xвместо-y-xдолжна содержать значение-xдолжна содержать значение после знака =ARGARGвместо опции-xwrongCloses #78