diff options
| author | Ian Cordasco <graffatcolmingov@gmail.com> | 2016-11-15 19:37:52 -0600 |
|---|---|---|
| committer | Ian Cordasco <graffatcolmingov@gmail.com> | 2016-11-15 19:56:18 -0600 |
| commit | c50b747a1a8b470574b5fc1016ee8cb598ab951e (patch) | |
| tree | 8ad32128af343f519ff54106770f6311b74bf3bc /src/flake8/main | |
| parent | 8e4905279cef9ac85725b56b4fab8e09cc981d5c (diff) | |
| download | flake8-c50b747a1a8b470574b5fc1016ee8cb598ab951e.tar.gz | |
Fix subtle reporting bug for default on plugins
When we refactored our decision process to fix #239, we broke a subtle
corner case where extensions that are not off-by-default are to be
reported.
This further refactors that logic and adds specific tests around it to
ensure it works as expected and doesn't regress.
Closes #257
Diffstat (limited to 'src/flake8/main')
| -rw-r--r-- | src/flake8/main/options.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/flake8/main/options.py b/src/flake8/main/options.py index 7e9b79e..9780378 100644 --- a/src/flake8/main/options.py +++ b/src/flake8/main/options.py @@ -63,7 +63,7 @@ def register_default_options(option_manager): ) add_option( - '--exclude', metavar='patterns', default=defaults.EXCLUDE, + '--exclude', metavar='patterns', default=','.join(defaults.EXCLUDE), comma_separated_list=True, parse_from_config=True, normalize_paths=True, help='Comma-separated list of files or directories to exclude.' @@ -102,7 +102,7 @@ def register_default_options(option_manager): ) add_option( - '--ignore', metavar='errors', default=defaults.IGNORE, + '--ignore', metavar='errors', default=','.join(defaults.IGNORE), parse_from_config=True, comma_separated_list=True, help='Comma-separated list of errors and warnings to ignore (or skip).' ' For example, ``--ignore=E4,E51,W234``. (Default: %default)', @@ -116,7 +116,7 @@ def register_default_options(option_manager): ) add_option( - '--select', metavar='errors', default=defaults.SELECT, + '--select', metavar='errors', default=','.join(defaults.SELECT), parse_from_config=True, comma_separated_list=True, help='Comma-separated list of errors and warnings to enable.' ' For example, ``--select=E4,E51,W234``. (Default: %default)', |