Merge pull request #553 from team-ocean/fix-precommit

bump flake8 and black versions

.pre-commit-config.yaml

@@ -1,12 +1,12 @@
 repos:
 -   repo: https://github.com/ambv/black
-    rev: 22.3.0
+    rev: 23.10.1
     hooks:
     - id: black
       language_version: python3
 
 -   repo: https://github.com/PyCQA/flake8
-    rev: 3.8.3
+    rev: 6.1.0
     hooks:
     - id: flake8
       args: ['--config=setup.cfg']

benchmarks/isoneutral_benchmark.py

@@ -47,7 +47,7 @@ def run():
 
         end = perf_counter()
 
-        logger.debug(f"Time step took {end-start}s")
+        logger.debug(f"Time step took {end - start}s")
 
 
 if __name__ == "__main__":

benchmarks/tdma_benchmark.py

@@ -66,7 +66,7 @@ def run():
 
         end = perf_counter()
 
-        logger.debug(f"Time step took {end-start}s")
+        logger.debug(f"Time step took {end - start}s")
 
 
 if __name__ == "__main__":

setup.cfg

@@ -12,4 +12,4 @@ tag_prefix = v
 exclude = veros/tools/filelock.py
 max-line-length = 120
 select = C,E,F,W,B,B950
-extend-ignore = E203, E501
+extend-ignore = E203,E501,W503

test/cli_test.py

@@ -61,7 +61,6 @@ def test_veros_run(runner, tmpdir):
         object.__setattr__(rs, "__locked__", False)
 
         try:
-
             result = runner.invoke(
                 veros.cli.veros_run.cli, [os.path.join(setup, f"{setup}.py"), "--backend", rs.backend]
             )

veros/cli/veros_resubmit.py

@@ -41,7 +41,7 @@ def unparse(args):
 
 def call_veros(cmd, name, n, runlen):
     identifier = f"{name}.{n:0>4}"
-    prev_id = f"{name}.{n-1:0>4}"
+    prev_id = f"{name}.{n - 1:0>4}"
     args = [
         "-s",
         "identifier",

veros/tools/filelock.py

@@ -98,6 +98,7 @@ def __str__(self):
 # Classes
 # ------------------------------------------------
 
+
 # This is a helper class which is returned by :meth:`BaseFileLock.acquire`
 # and wraps the lock to make sure __enter__ is not called twice when entering
 # the with statement.
@@ -294,7 +295,6 @@ def release(self, force=False):
             every case.
         """
         with self._thread_lock:
-
             if self.is_locked:
                 self._lock_counter -= 1