Merge pull request #62 from cvxgrp/pre

pre-commit

Author: tschm

.pre-commit-config.yaml

@@ -5,23 +5,20 @@ repos:
       - id: check-toml
       - id: end-of-file-fixer
       - id: trailing-whitespace
-
-  - repo: https://github.com/psf/black
-    rev: 24.10.0
-    hooks:
-      - id: black
+      - id: check-yaml
 
   - repo: https://github.com/charliermarsh/ruff-pre-commit
     rev: 'v0.8.4'
     hooks:
       - id: ruff
         args: [ --fix, --exit-non-zero-on-fix ]
+      # Run the formatter
+      - id: ruff-format
 
   - repo: https://github.com/igorshubovych/markdownlint-cli
     rev: v0.43.0
     hooks:
       - id: markdownlint-fix
-        args: [ "--ignore", "book/**/*.md" ]
 
   - repo: https://github.com/asottile/pyupgrade
     rev: v3.19.1
@@ -36,16 +33,16 @@ repos:
       - id: check-github-workflows
         args: ["--verbose"]
 
-  #- repo: https://github.com/python-poetry/poetry
-  #  rev: '1.6.1'  # add version here
-  #  hooks:
-  #    - id: poetry-check
-  #    - id: poetry-lock
-
   - repo: https://github.com/Lucas-C/pre-commit-hooks
     rev: v1.5.5
     hooks:
         - id: insert-license
           files: ^(cvx)
           args:
             ['--license-filepath', 'copyright.txt', '--no-extra-eol']
+
+  - repo: https://github.com/rhysd/actionlint
+    rev: v1.7.5
+    hooks:
+      - id: actionlint
+        args: [-ignore, SC]

cvx/cli/smallest_eigenvalue.py

@@ -31,10 +31,7 @@ def smallest_ev(bson_file) -> Dict[str, float]:
     poetry run smallest-eigenvalue cli/data/test.bson
     """
 
-    eigenvalues = {
-        key: np.min(np.linalg.eigh(matrix)[0])
-        for key, matrix in read_bson(bson_file).items()
-    }
+    eigenvalues = {key: np.min(np.linalg.eigh(matrix)[0]) for key, matrix in read_bson(bson_file).items()}
 
     for key, ev in eigenvalues.items():
         print(key, ev)

pyproject.toml

@@ -27,18 +27,16 @@ dev = [
 weather = "cvx.cli.weather:main"
 smallest-eigenvalue = "cvx.cli.smallest_eigenvalue:main"
 
-#[build-system]
-#requires = ["poetry>=1.6.0"]
-#build-backend = "poetry.core.masonry.api"
-
 [tool.ruff]
-select = ["E", "F", "I"]
 line-length = 120
 target-version = "py310"
 exclude = [
     "*__init__.py"
 ]
 
+[tool.ruff.lint]
+select = ["E", "F", "I"]
+
 [build-system]
 requires = ["hatchling"]
 build-backend = "hatchling.build"