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
Fix readme file on Extending Bandit on list things #534
Conversation
@aurel10 Please rebase and resolve conflicts, thanks! |
… Extending Bandit
dc613cb
to
76ca274
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some nit changes, otherwise good to go.
README.rst
Outdated
- `manager`: An instance of `bandit manager` | ||
- `fileobj`: output file object | ||
- `manager`: this is an instance of `bandit manager` | ||
- `fileobj`: the output file object, which may be sys.stdout |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: there are two spaces after the ,
README.rst
Outdated
manager, fileobj, sev_level, conf_level, lines=-1 | ||
- `manager`: An instance of `bandit manager` | ||
- `fileobj`: output file object | ||
- `manager`: this is an instance of `bandit manager` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- `manager`: this is an instance of `bandit manager` | |
- `manager`: an instance of `bandit manager` |
README.rst
Outdated
- `sev_level` : Filtering severity level | ||
- `conf_level`: Filtering confidence level | ||
- `lines=-1`: Number of lines to report | ||
- `lines=-1`: indicate the number of lines to report |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- `lines=-1`: indicate the number of lines to report | |
- `lines=-1`: number of lines to report |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fix readme file on Extending Bandit on list things;
manager, fileobj, sev_level, conf_level, lines=-1
they were not listed on Markdown format