Merge pull request #132 from Kuniwak/support-format

Support --format option

Author: Kuniwak

vint/linting/cli.py

@@ -74,6 +74,7 @@ def _build_argparser(self):
         parser.add_argument('-c', '--color', action='store_true', help='colorize output when possible')
         parser.add_argument('-j', '--json', action='store_true', help='output json style')
         parser.add_argument('-t', '--stat', action='store_true', help='output statistic info')
+        parser.add_argument('-f', '--format', help='set output format')
         parser.add_argument('files', nargs='*', help='file or directory path to lint')
 
         return parser

vint/linting/config/config_cmdargs_source.py

@@ -20,6 +20,7 @@ def _build_config_dict(self, env):
         config_dict = self._normalize_verbose(env, config_dict)
         config_dict = self._normalize_severity(env, config_dict)
         config_dict = self._normalize_max_violations(env, config_dict)
+        config_dict = self._normalize_format(env, config_dict)
 
         return config_dict
 
@@ -54,6 +55,10 @@ def _normalize_max_violations(self, env, config_dict):
         return self._pass_config_by_key('max-violations', env, config_dict)
 
 
+    def _normalize_format(self, env, config_dict):
+        return self._pass_config_by_key('format', env, config_dict)
+
+
     def _normalize_severity(self, env, config_dict):
         env_cmdargs = env['cmdargs']
         config_dict_cmdargs = config_dict['cmdargs']

vint/linting/formatter/formatter.py

@@ -23,12 +23,12 @@ def __init__(self, config_dict):
         else:
             cmdargs = {}
 
-        if 'format' in cmdargs:
+        if 'format' in cmdargs and cmdargs['format'] is not None:
             self._format = cmdargs['format']
         else:
             self._format = DEFAULT_FORMAT
 
-        if 'color' in cmdargs:
+        if 'color' in cmdargs and cmdargs['color'] is not None:
             self._should_be_colorized = cmdargs['color']
         else:
             self._should_be_colorized = False