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.

Sign up for GitHub

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

Jump to bottom

SubcommandRequired in yml #593

Closed
laishulu opened this issue Jul 23, 2016 · 10 comments
Closed

SubcommandRequired in yml #593

laishulu opened this issue Jul 23, 2016 · 10 comments
Assignees
@kbknapp
Labels
C-bug Category: Updating dependencies

Comments

@laishulu
Copy link

laishulu commented Jul 23, 2016 •
edited

I can NOT run if I have the following line in the yml file, what's the correct way to config this?

setting: "SubcommandRequired"
@kbknapp
Copy link
Member

kbknapp commented Jul 23, 2016

Thanks! I'll put this in the queue, but it should be an easy fix.

@kbknapp kbknapp added C-bug Category: Updating dependencies P2: need to have A-help Area: documentation, including docs.rs, readme, examples, etc... labels Jul 23, 2016
@kbknapp kbknapp self-assigned this Jul 23, 2016
@kbknapp kbknapp added T: RFC / question and removed A-help Area: documentation, including docs.rs, readme, examples, etc... C: settings C-bug Category: Updating dependencies labels Jul 23, 2016
@kbknapp
Copy link
Member

kbknapp commented Jul 23, 2016

I'm sorry, I was confused about what you were asking.

In your YAML file, putting the following will work just fine:

settings:
    - SubcommandRequired

@laishulu
Copy link
Author

Still got the following result:

thread 'main' panicked at 'unknown AppSetting found in YAML file', ../src/libcore/option.rs:699

@kbknapp
Copy link
Member

kbknapp commented Jul 23, 2016

Ah found the typo bug!

Thanks! 👍

@kbknapp kbknapp added C-bug Category: Updating dependencies P2: need to have and removed T: RFC / question labels Jul 23, 2016
@kbknapp kbknapp mentioned this issue Jul 23, 2016
Closed
@kbknapp
Copy link
Member

kbknapp commented Jul 23, 2016

#594 fixes this

@kbknapp
Copy link
Member

kbknapp commented Jul 23, 2016

Correction #595 fixes this

@homu homu closed this as completed in fc72cdf Jul 23, 2016
@laishulu
Copy link
Author

Is it pushed to crates as a new version?
Or I should install it directly from git repo?

@kbknapp
Copy link
Member

kbknapp commented Jul 24, 2016

I will upload v2.9.3 to crates.io when I get home tonight. I'll also post here once the new version is uploaded 😉

@laishulu
Copy link
Author

great!

@kbknapp
Copy link
Member

kbknapp commented Jul 24, 2016

v2.9.3 is out on crates.io

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kbknapp kbknapp

Labels
C-bug Category: Updating dependencies
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@laishulu @kbknapp