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

Make 'target' arg optional and let it be defined in the ini file #274

Closed
ericwb opened this issue Apr 27, 2018 · 2 comments
Closed

Make 'target' arg optional and let it be defined in the ini file #274

ericwb opened this issue Apr 27, 2018 · 2 comments

Comments

@ericwb
Copy link
Member

ericwb commented Apr 27, 2018

settings that are read by bandit from ini files are already almost complete for most projects (excludes, tests and skips) but are missing 'target' that must be always specified explicitly.

Adding it to CLI arguments that could be read from ini file would ease bandit configuration as we could store its ini settings in the tox.ini itself (almost like flake8 allows) and run it as "bandit --ini tox.ini"

Later we could add autodetection of bandit section in the tox.ini too.

@ericwb ericwb closed this as completed Apr 27, 2018
@bittner
Copy link
Contributor

bittner commented Mar 21, 2019

Why was this closed?

Both the optional target argument and the tox.ini file configuration section are valuable enhancements.

Can we reopen this issue?

@ericwb
Copy link
Member Author

ericwb commented Mar 25, 2019

@bittner This issue was transferred incorrectly from Launchpad in the openstack namespace. The bug was fixed in LP. See https://bugs.launchpad.net/bandit/+bug/1730307

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants