Merge pull request #64 from seddonym/black-check

Black check

src/importlinter/adapters/building.py

@@ -1,5 +1,4 @@
 import grimp  # type: ignore
-
 from importlinter.application.ports import building as ports
 from importlinter.domain.ports.graph import ImportGraph
 

src/importlinter/contracts/forbidden.py

@@ -30,8 +30,8 @@ def check(self, graph: ImportGraph) -> ContractCheck:
         is_kept = True
         invalid_chains = []
 
-        removed_imports = helpers.pop_imports(  # type: ignore
-            graph, self.ignore_imports if self.ignore_imports else []
+        removed_imports = helpers.pop_imports(
+            graph, self.ignore_imports if self.ignore_imports else []  # type: ignore
         )
 
         self._check_all_modules_exist_in_graph(graph)

src/importlinter/contracts/independence.py

@@ -30,15 +30,13 @@ def check(self, graph: ImportGraph) -> ContractCheck:
         is_kept = True
         invalid_chains = []
 
-        removed_imports = helpers.pop_imports(  # type: ignore
-            graph, self.ignore_imports if self.ignore_imports else []
+        removed_imports = helpers.pop_imports(
+            graph, self.ignore_imports if self.ignore_imports else []  # type: ignore
         )
 
         self._check_all_modules_exist_in_graph(graph)
 
-        for subpackage_1, subpackage_2 in permutations(  # type: ignore
-            self.modules, r=2
-        ):
+        for subpackage_1, subpackage_2 in permutations(self.modules, r=2):  # type: ignore
             subpackage_chain_data = {
                 "upstream_module": subpackage_2.name,
                 "downstream_module": subpackage_1.name,

src/importlinter/contracts/layers.py

@@ -59,8 +59,8 @@ def check(self, graph: ImportGraph) -> ContractCheck:
         self._validate_containers()
 
         direct_imports_to_ignore = self.ignore_imports if self.ignore_imports else []
-        removed_imports = helpers.pop_imports(  # type: ignore
-            graph, direct_imports_to_ignore
+        removed_imports = helpers.pop_imports(
+            graph, direct_imports_to_ignore  # type: ignore
         )
 
         for container in self.containers:  # type: ignore

src/importlinter/domain/ports/graph.py

@@ -27,9 +27,7 @@ def find_shortest_chain(self, importer: str, imported: str) -> Optional[Tuple[st
         raise NotImplementedError
 
     @abc.abstractmethod
-    def find_shortest_chains(
-        self, importer: str, imported: str,
-    ) -> Set[Tuple[str, ...]]:
+    def find_shortest_chains(self, importer: str, imported: str) -> Set[Tuple[str, ...]]:
         """
         Find the shortest import chains that exist between the importer and imported, and
         between any modules contained within them. Only one chain per upstream/downstream pair

tests/adapters/building.py

@@ -9,6 +9,7 @@ class FakeGraphBuilder(GraphBuilder):
     To use, call inject_graph with the graph you wish to inject, ahead
     of when the builder would be called.
     """
+
     def build(
         self, root_package_name: str, include_external_packages: bool = False
     ) -> ImportGraph:

tox.ini

@@ -27,9 +27,11 @@ commands =
 
 [testenv:check]
 deps =
-    flake8
-    mypy
+    black==19.3b0
+    flake8==3.7.8
+    mypy==0.730
 commands =
+    black --check src tests
     flake8 src tests setup.py
     mypy src/importlinter tests
     lint-imports