poetry update

Makefile

@@ -21,6 +21,12 @@ install:
 update:
 	poetry update
 
+check:
+	poetry check
+	make typecheck
+	make lint
+
+
 test: typecheck
 	echo "Testing with WRITE_BUCKET=$(WRITE_BUCKET)"
 	poetry run pytest $(TEST_README) --cov --cov-report xml:coverage.xml
@@ -34,6 +40,10 @@ test-long:
 typecheck:
 	poetry run mypy $(PROJECT) tests
 
+lint:
+	poetry run black $(PROJECT) tests
+	poetry run flake8 $(PROJECT) tests
+
 coverage:
 	echo "Using WRITE_BUCKET=$(WRITE_BUCKET) | SKIP_LONG_TESTS=$(SKIP_LONG_TESTS)"
 	poetry run pytest --cov --cov-report html:coverage_html

pyproject.toml

@@ -27,6 +27,7 @@ universal-pathlib = "^0.1.1"
 urllib3 = "<2"
 
 [tool.poetry.group.dev.dependencies]
+black = "^23.9.1"
 flake8 = "^6.0.0"
 mypy = "^1.4.1"
 pytest-cov = "^4.0.0"

quiltplus/package.py

@@ -2,7 +2,6 @@
 
 import logging
 import shutil
-from pathlib import Path
 
 from quilt3 import Package  # type: ignore
 from quiltcore import Builder, Changes, Manifest
@@ -63,32 +62,32 @@ async def get(self, opts: dict = {}):
         dest = self.check_dir_arg(opts)
         logging.debug(f"get dest={dest}: {opts}")
         man = self.remote_man()
-        rc = self.volume.put(man)  # TODO: update self.tag
+        self.volume.put(man)  # TODO: update self.tag
         files = self.local_files()
         return [f"file://{fn}" for fn in files]
-    
+
     #
     # Create/Revise Package
     #
 
-    def unchanged(self) -> bool: # pragma: no cover
+    def unchanged(self) -> bool:  # pragma: no cover
         if not hasattr(self, "changes"):
             return True
         if not isinstance(self.changes, Changes):
             return True
 
         return len(self.changeset()) == 0
 
-    def changeset(self) -> Changes: # pragma: no cover
+    def changeset(self) -> Changes:  # pragma: no cover
         vpath = self.volume.path
         if self.unchanged():
             self.changes = Changes(vpath)
         if self.changes.path == vpath:
             return self.changes
-        
+
         raise ValueError(f"{self.ERROR_VOL}: {self.changes.path} != {vpath}")
 
-    def commit(self, **kwargs) -> Manifest: # pragma: no cover
+    def commit(self, **kwargs) -> Manifest:  # pragma: no cover
         """
         Create manifest.
         Store in the local registry.
@@ -111,9 +110,7 @@ async def push(self, q: Package, opts: dict):
         kwargs = {
             self.K_REG: self.registry,
             self.K_FORCE: not opts.get(self.K_FAIL, False),
-            self.K_MSG: opts.get(
-                self.K_MSG, f"{__name__} {self.Now()} @ {opts}"
-            ),
+            self.K_MSG: opts.get(self.K_MSG, f"{__name__} {self.Now()} @ {opts}"),
         }
         logging.debug(f"push dest={dest}: {opts}\n{kwargs}")
         q.set_dir(".", dest)
@@ -157,7 +154,7 @@ async def local_pkg(self):
 
     async def remote_pkg(self):
         return (await self.browse()) or Package()
-    
+
     def delete(self):  # remove local cache
         return shutil.rmtree(self.last_path)
 

quiltplus/type.py

@@ -1,7 +1,4 @@
-from datetime import datetime
-
 from quiltcore import Resource
-from tzlocal import get_localzone
 from un_yaml import UnUri  # type: ignore
 
 

setup.cfg

@@ -0,0 +1,2 @@
+[flake8]
+max-line-length = 127

tests/test_push.py

@@ -45,7 +45,7 @@ async def test_push_patch(tmpdir: Path):
 
 
 @pytest.mark.skip("Not fully implemented")
-def test_push_core_commit(tmpdir: Path): # pragma: no cover
+def test_push_core_commit(tmpdir: Path):  # pragma: no cover
     name = "test_push_core_commit"
     pkg = get_unique_pkg(name)
     cache = tmpdir / name