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

Allow users to specify own options #92

Open
sbeyer opened this issue Mar 17, 2017 · 1 comment
Open

Allow users to specify own options #92

sbeyer opened this issue Mar 17, 2017 · 1 comment
Labels
feature Feature request
Milestone

Comments

@sbeyer
Copy link
Member

sbeyer commented Mar 17, 2017

Users should be able to set up an array for the option parser
and provide it to bandit::run to allow additional options (for
example --extended-tests or something might be a use-case).
Those options must be accessible within all tests.

@sbeyer sbeyer added the feature Feature request label Mar 17, 2017
@sbeyer sbeyer added this to the 3.0 Release milestone Mar 17, 2017
sbeyer added a commit that referenced this issue Mar 17, 2017
This is not the best solution to the problem that the
users may want additional options and tackle them. But
it is a solution to the fact that users may want usage
information when they use invalid options or arguments.

Hence it is merged as an intermediate solution.
A better solution is described in #92.
@sbeyer
Copy link
Member Author

sbeyer commented Mar 19, 2017

Also let bandit be more helpful and show users which of their used options are unknown, etc.

sbeyer added a commit that referenced this issue Mar 19, 2017
It could be helpful for the user to see when she used options
that are unknown. However, bandit's usage is simple enough to
assume that the usage screen is helpful enough.

Also printing unknown options may be confusing if user code adds
custom options (which is a hacky thing, so far...)
Hence, this commit removes the TODO marker.

As soon as there is a nice way to add custom options to bandit (see #92),
one should add such output.
sbeyer added a commit that referenced this issue Mar 19, 2017
It could be helpful for the user to see when she used options
that are unknown. However, bandit's usage is simple enough to
assume that the usage screen is helpful enough.

Also printing unknown options may be confusing if user code adds
custom options (which is a hacky thing, so far...)
Hence, this commit removes the TODO marker.

As soon as there is a nice way to add custom options to bandit (see #92),
one should add such output.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Feature request
Projects
None yet
Development

No branches or pull requests

1 participant