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

Add support for options _following_ positional parameters #130

Closed
remkop opened this issue May 25, 2017 · 3 comments
Closed

Add support for options _following_ positional parameters #130

remkop opened this issue May 25, 2017 · 3 comments
Labels
type: enhancement ✨
Milestone
2.0.0

Comments

@remkop
Copy link
Owner

remkop commented May 25, 2017

For example

ansible somehost -m ping
@remkop remkop modified the milestones: 1.?.0 polishing, backlog May 25, 2017
@remkop remkop mentioned this issue Jul 18, 2017
Closed
@remkop remkop mentioned this issue Sep 23, 2017
Closed
@remkop remkop modified the milestones: backlog, 2.0.0 Sep 28, 2017
@kbruegge
Copy link

This would indeed be a very welcome addition. Keep up the good work. 👍

@remkop
Copy link
Owner Author

remkop commented Sep 28, 2017

Thanks for the positive feedback!
Note that I'm going to have to change picocli's current default to be able to do this.

See #192. I'm not happy about the disruption this may cause but I don't see any other way forward.

@remkop remkop mentioned this issue Sep 30, 2017
Closed
remkop added a commit that referenced this issue Oct 12, 2017
@remkop
#130 support mixing positional parameters with options on the command…
faf8311 
… line
remkop added a commit that referenced this issue Oct 12, 2017
@remkop
#130 improved tracing for positional parameters
7a6ae6b
remkop added a commit that referenced this issue Oct 12, 2017
@remkop
#130 fix test to be OS-agnostic
aa2a60c
@remkop
Copy link
Owner Author

remkop commented Oct 13, 2017

The implementation for this is done and pushed to master. Still todo: documentation updates and potentially more unit tests.

remkop added a commit that referenced this issue Oct 15, 2017
@remkop
#130 Update user manual and add tests for mixing positional parameter…
117e3b3 
…s with options on the command line
@remkop remkop closed this as completed Oct 15, 2017
remkop added a commit that referenced this issue Oct 16, 2017
@remkop
#130 un-ignore script test mixing positional params with options
8aecfb0
@remkop remkop mentioned this issue Oct 29, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: enhancement ✨
Projects
None yet
Milestone
2.0.0
Development

No branches or pull requests
2 participants
@kbruegge @remkop