Merge pull request #286 from authzed/284-use-ruff

Use ruff instead of flake8/black/etc

Author: tstirrat15

.github/workflows/lint.yaml

@@ -30,27 +30,10 @@ jobs:
       - uses: "actions/setup-python@v6"
         with:
           python-version: "3.10"
-      - name: "Setup Python Environment"
-        run: "pip install -U pip virtualenv"
-      - name: "Install Dependencies"
-        run: |
-          virtualenv ~/.cache/virtualenv/authzedpy
-          source ~/.cache/virtualenv/authzedpy/bin/activate
-          pip install poetry
-          poetry env info
-          poetry install --only dev
-      - name: "Pyflakes"
-        run: |
-          source ~/.cache/virtualenv/authzedpy/bin/activate
-          find . -name "*.py" | grep -v "_pb2" | xargs pyflakes
-      - name: "Blacken"
-        run: |
-          source ~/.cache/virtualenv/authzedpy/bin/activate
-          black --check --diff .
-      - name: "Isort"
-        run: |
-          source ~/.cache/virtualenv/authzedpy/bin/activate
-          find . -name "*.py" | grep -v "_pb2" | xargs isort --check --diff
+      # Installs ruff and runs check
+      - uses: "astral-sh/ruff-action@v3"
+      # Check formatting as well
+      - run: "ruff format --check --diff"
   codeql:
     name: "Analyze with CodeQL"
     runs-on: "ubuntu-latest"

poetry.lock

@@ -29,21 +29,24 @@ googleapis-common-protos = "^1.65.0"
 protovalidate = ">=0.7.1,<1.1.0"
 
 [tool.poetry.group.dev.dependencies]
-black = ">=23.3,<26.0"
 grpc-stubs = "^1.53"
 grpcio-tools = ">=1.63,<1.72"
-isort = ">=5.6.4,<7.0.0"
 mypy = "1.18.1"
 mypy-protobuf = "3.6.0"
 mock = "^5.1.0"
-pyflakes = "^3.0.1"
 pytest = ">=7.1.3,<9.0.0"
 pytest-asyncio = ">=0.21,<1.3"
 types-protobuf = ">=5.26,<7.0"
 typing-extensions = ">=3.7.4,<5"
+ruff = "^0.13.0"
 
-[tool.black]
-exclude = '(^\.(.*)|^(.*)_pb2(_grpc)?\.py)'
+[tool.ruff]
+exclude = [
+"*_pb2.py",
+"*_grpc.py",
+"*_pb2.pyi",
+"*_grpc.pyi",
+]
 line-length = 100
 
 [tool.mypy]
@@ -59,14 +62,6 @@ log_level = "debug"
 minversion = "6.0"
 asyncio_mode = "auto"
 
-[tool.isort]
-ensure_newline_before_comments = true
-force_grid_wrap = 0
-include_trailing_comma = true
-line_length = 100
-multi_line_output = 3
-use_parentheses = true
-
 [build-system]
 build-backend = "poetry.core.masonry.api"
 requires = ["poetry-core>=1.0.0"]

pyproject.toml

@@ -328,7 +328,7 @@ async def test_bulk_export_import(client):
 
     # do bulk import
     reqs = [BulkImportRelationshipsRequest(relationships=rels)]
-    import_rels = await maybe_await(empty_client.BulkImportRelationships(((req for req in reqs))))
+    import_rels = await maybe_await(empty_client.BulkImportRelationships((req for req in reqs)))
     assert import_rels.num_loaded == 4
 
     # validate all relationships were imported