pep8 compliance (#257)

add pep8 check to continuous integration tests and bring codebase into compliance

automatic commit by git-black, original commits:
  0e01064

core/dbt/deprecations.py

@@ -90,9 +90,7 @@ class AdapterDeprecationWarning(DBTDeprecation):
 def warn(name, *args, **kwargs):
     if name not in deprecations:
         # this should (hopefully) never happen
-        raise RuntimeError(
-            "Error showing deprecation warning: {}".format(name)
-        )
+        raise RuntimeError("Error showing deprecation warning: {}".format(name))
 
     deprecations[name].show(*args, **kwargs)
 

core/dbt/main.py

@@ -199,7 +199,7 @@ def track_run(task):
     dbt.tracking.track_invocation_start(config=task.config, args=task.args)
     try:
         yield
-        dbt.tracking.track_invocation_end(
+        dbt.tracking.track_invocation_end(config=task.config, args=task.args, result_type="ok")
             config=task.config, args=task.args, result_type="ok"
         )
     except (NotImplementedException,
@@ -260,7 +260,7 @@ def _build_base_subparser():
     )
 
     base_subparser.add_argument(
-        '--profiles-dir',
+        "--profiles-dir",
         default=None,
         dest="sub_profiles_dir",  # Main cli arg precedes subcommand
         type=str,
@@ -272,7 +272,7 @@ def _build_base_subparser():
     )
 
     base_subparser.add_argument(
-        '--profile',
+        "--profile",
         required=False,
         type=str,
         help='''
@@ -282,7 +282,7 @@ def _build_base_subparser():
 
     base_subparser.add_argument(
         '-t',
-        '--target',
+        "--target",
         default=None,
         type=str,
         help='''
@@ -478,7 +478,7 @@ def _build_snapshot_subparser(subparsers, base_subparser):
         ''',
     )
     sub.add_argument(
-        '--threads',
+        "--threads",
         type=int,
         required=False,
         help='''

core/dbt/task/clean.py

@@ -28,8 +28,7 @@ def __is_protected_path(self, path):
         protected_paths = self.config.model_paths + \
             self.config.test_paths + ['.']
         protected_abs_paths = [os.path.abspath(p) for p in protected_paths]
-        return abs_path in set(protected_abs_paths) or \
-            self.__is_project_path(abs_path)
+        return abs_path in set(protected_abs_paths) or self.__is_project_path(abs_path)
 
     def run(self):
         """

core/dbt/tracking.py

@@ -402,7 +402,7 @@ def track_invocation_end(
     )
 
 
-def track_invalid_invocation(
+def track_invalid_invocation(config=None, args=None, result_type=None):
         config=None, args=None, result_type=None
 ):
     assert active_user is not None, \