Merge pull request #28 from hackebrot/fix-ci-checks

Fix CI checks 🤖

Author: hackebrot

src/labels/cli.py

@@ -174,7 +174,9 @@ def sync_cmd(
     repository = Repository(owner, repo)
 
     try:
-        remote_labels = {l.name: l for l in context.client.list_labels(repository)}
+        remote_labels = {
+            label.name: label for label in context.client.list_labels(repository)
+        }
     except LabelsException as exc:
         click.echo(str(exc), err=True)
         sys.exit(1)
@@ -261,21 +263,21 @@ def dryrun_echo(
     """Print information about how labels would be updated on sync."""
 
     if labels_to_delete:
-        click.echo(f"This would delete the following labels:")
+        click.echo("This would delete the following labels:")
         for name in labels_to_delete:
             click.echo(f"  - {name}")
 
     if labels_to_update:
-        click.echo(f"This would update the following labels:")
+        click.echo("This would update the following labels:")
         for name in labels_to_update:
             click.echo(f"  - {name}")
 
     if labels_to_create:
-        click.echo(f"This would create the following labels:")
+        click.echo("This would create the following labels:")
         for name in labels_to_create:
             click.echo(f"  - {name}")
 
     if labels_to_ignore:
-        click.echo(f"This would NOT modify the following labels:")
+        click.echo("This would NOT modify the following labels:")
         for name in labels_to_ignore:
             click.echo(f"  - {name}")

src/labels/github.py

@@ -84,7 +84,7 @@ def list_labels(self, repo: Repository) -> List[Label]:
 
         while next_page is not None:
 
-            logger.debug(f"Requesting next page of labels")
+            logger.debug("Requesting next page of labels")
             response = self.session.get(next_page["url"], headers=headers)
 
             if response.status_code != 200:

src/labels/utils.py

@@ -27,7 +27,7 @@ def load_repository_info(remote_name: str = "origin") -> typing.Optional[Reposit
     )
 
     if proc.returncode != 0:
-        logger.debug(f"Error running git remote get-url.")
+        logger.debug("Error running git remote get-url.")
         return None
 
     remote_url = proc.stdout.strip()

tests/test_github.py

@@ -42,7 +42,7 @@ def test_list_labels(client: Client, repo: Repository) -> None:
         },
     ]
 
-    assert [l.params_dict for l in labels] == expected_params
+    assert [label.params_dict for label in labels] == expected_params
 
 
 @pytest.mark.usefixtures("mock_list_labels_paginated")
@@ -68,7 +68,7 @@ def test_list_labels_pagination(client: Client, repo: Repository) -> None:
         },
     ]
 
-    assert [l.params_dict for l in labels] == expected_params
+    assert [label.params_dict for label in labels] == expected_params
 
 
 @pytest.mark.usefixtures("mock_get_label")

tests/test_io.py

@@ -24,7 +24,7 @@ def test_load_labels(labels_file_load: str, labels: typing.List[Label]) -> None:
     """Test that read_lables() correctly reads the TOML file and returns a
     mapping of names to Label instances.
     """
-    want = {l.name: l for l in labels}
+    want = {label.name: label for label in labels}
     got = read_labels(labels_file_load)
 
     assert got == want

tox.ini

@@ -16,8 +16,10 @@ commands = pytest -v --cov labels {posargs:tests}
 
 [testenv:flake8]
 deps = flake8
-commands = flake8
+commands = flake8 {toxinidir}/setup.py {toxinidir}/src/ {toxinidir}/tests/
 
 [testenv:mypy]
-deps = mypy
-commands = mypy {toxinidir}/src/
+deps =
+    mypy
+    types-requests
+commands = mypy {toxinidir}/setup.py {toxinidir}/src/ {toxinidir}/tests/