Merge pull request #103 from bento-platform/lint/black

lint: set up black linter/formatter

.github/workflows/lint.yml

@@ -2,18 +2,24 @@ name: Lint
 on:
   push:
     branches:
-      - master
+      - main
   pull_request:
 jobs:
   lint:
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: [ "3.10" ]
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5
         name: Set up Python
         with:
-          python-version: "3.10"
-      - name: Install flake8
-        run: python -m pip install flake8
-      - name: Run linter
-        run: flake8 ./chord_drs ./tests --exclude=migrations
+          python-version: ${{ matrix.python-version }}
+      - name: Install poetry
+        run: pip install poetry
+      - name: Install dependencies
+        run: poetry install
+      - name: Lint
+        run: |
+          poetry run black --check ./chord_drs ./tests

chord_drs/app.py

@@ -43,36 +43,41 @@
         drs_compat=True,
         logger=application.logger,
         authz=authz_middleware,
-    ))
+    ),
+)
 application.register_error_handler(
     BadRequest,
     flask_errors.flask_error_wrap(
         flask_errors.flask_bad_request_error,
         drs_compat=True,
         authz=authz_middleware,
-    ))
+    ),
+)
 application.register_error_handler(
     Forbidden,
     flask_errors.flask_error_wrap(
         flask_errors.flask_forbidden_error,
         drs_compat=True,
         authz=authz_middleware,
-    ))
+    ),
+)
 application.register_error_handler(
     NotFound,
     lambda e: flask_errors.flask_error_wrap(
         flask_errors.flask_not_found_error,
         str(e),
         drs_compat=True,
         authz=authz_middleware,
-    )(e))
+    )(e),
+)
 application.register_error_handler(
     RequestedRangeNotSatisfiable,
     flask_errors.flask_error_wrap(
         flask_errors.flask_range_not_satisfiable_error,
         drs_compat=True,
         authz=authz_middleware,
-    ))
+    ),
+)
 
 # Attach the database to the application and run migrations if needed
 db.init_app(application)

chord_drs/backends/base.py

@@ -14,5 +14,6 @@ class FakeBackend(Backend):
     """
     For the tests
     """
+
     def save(self, current_location: str, filename: str) -> str:
         return current_location

chord_drs/backends/local.py

@@ -14,6 +14,7 @@ class LocalBackend(Backend):
     by this service. Lives on the current filesystem, in a directory
     specified by the DATA var env, the default being in ~/chord_drs_data
     """
+
     def __init__(self):
         self.base_location = Path(current_app.config["SERVICE_DATA"])
         # We can use mkdir, since resolve has been called in config.py

chord_drs/backends/minio.py

@@ -12,10 +12,10 @@
 class MinioBackend(Backend):
     def __init__(self, resource=None):
         self.minio = resource or boto3.resource(
-            's3',
+            "s3",
             endpoint_url=current_app.config["MINIO_URL"],
             aws_access_key_id=current_app.config["MINIO_USERNAME"],
-            aws_secret_access_key=current_app.config["MINIO_PASSWORD"]
+            aws_secret_access_key=current_app.config["MINIO_PASSWORD"],
         )
 
         self.bucket = self.minio.Bucket(current_app.config["MINIO_BUCKET"])

chord_drs/config.py

@@ -32,10 +32,8 @@ def _get_from_environ_or_fail(var: str) -> str:
 # when deployed inside chord_singularity, DATABASE will be set
 BASEDIR = os.environ.get("DATABASE", APP_DIR.parent)
 SERVICE_DATA: str = str(
-    Path(os.environ.get("DATA", os.path.join(Path.home(), "chord_drs_data")))
-    .expanduser()
-    .absolute()
-    .resolve())
+    Path(os.environ.get("DATA", os.path.join(Path.home(), "chord_drs_data"))).expanduser().absolute().resolve()
+)
 
 # Authorization variables
 AUTHZ_ENABLED = os.environ.get("AUTHZ_ENABLED", "true").strip().lower() in TRUTH_VALUES