Merge pull request swiftlang#8238 from hughbe/python-lint-exitcode

Integrate python-lint into validation tests

Author: hughbe

utils/python-lint

@@ -24,14 +24,16 @@ def main():
     if flake8_result != 0:
         print("Missing modules flake8 or flake8-import-order. Please be sure "
               "to install these python packages before linting.")
+        sys.exit(0)
 
     utils_directory = os.path.dirname(os.path.abspath(__file__))
     parent_directory = os.path.dirname(utils_directory)
-    subprocess.call(
+    linting_result = subprocess.call(
       [sys.executable, "-m", "flake8"] + sys.argv[1:],
       cwd=parent_directory,
       universal_newlines=True
     )
+    sys.exit(linting_result)
 
 
 if __name__ == '__main__':

utils/swift_build_support/tests/test_workspace.py

@@ -50,7 +50,8 @@ def test_workspace(self):
 
 class ComputeBuildSubdirTestCase(unittest.TestCase):
 
-    def create_basic_args(self, generator, variant, assertions, enable_asan=False):
+    def create_basic_args(self, generator, variant, assertions,
+                          enable_asan=False):
         return argparse.Namespace(
             cmake_generator=generator,
             cmark_build_variant=variant,
@@ -139,7 +140,8 @@ def test_all_combinations_are_unique(self):
 
         def generate():
             for c in productions:
-                args = argparse.Namespace(cmake_generator="Ninja", enable_asan=False)
+                args = argparse.Namespace(cmake_generator="Ninja",
+                                          enable_asan=False)
                 for key, val in zip(keys, c):
                     setattr(args, key, val)
                 yield compute_build_subdir(args)

validation-test/Python/python-lint.swift

@@ -0,0 +1 @@
+// RUN: %{python} %utils/python-lint