Merge pull request #27 from simonsobs/dev

Format code with `ruff`

.vscode/settings.json

@@ -10,7 +10,6 @@
   "python.testing.pytestArgs": ["tests"],
   "python.testing.unittestEnabled": false,
   "python.testing.pytestEnabled": true,
-  "isort.check": true,
   "markdownlint.config": {
     "MD033": false,
     "MD036": false,

pyproject.toml

@@ -54,9 +54,32 @@ addopts = "--doctest-modules"
 doctest_optionflags = ["ELLIPSIS"]
 norecursedirs = "build"
 
-[tool.black]
-skip-string-normalization = true
-target_version = ['py310', 'py311', 'py312']
+[tool.ruff]
+line-length = 88
+
+[tool.ruff.format]
+quote-style = "preserve"
+
+[tool.ruff.lint]
+select = [
+  "E", # pycodestyle errors
+  "W", # pycodestyle warnings
+  "F", # pyflakes
+  "I", # isort
+]
+ignore = [
+  "E203", # Whitespace before '{symbol}'
+  "E265", # Block comment should start with #
+  "E266", # Too many leading # before block comment
+  "E501", # Line too long ({width} > {limit})
+  "E741", # Ambiguous variable name '{name}'
+]
+
+[tool.ruff.lint.mccabe]
+max-complexity = 20
+
+[tool.ruff.lint.isort]
+known-first-party = ["nextline", "nextlinegraphql"]
 
 [tool.coverage.run]
 source_pkgs = ["nextline_alert", "tests"]

src/nextline_alert/emitter.py

@@ -1,6 +1,7 @@
 from logging import getLogger
 
 import httpx
+
 from nextline.plugin.spec import Context, hookimpl
 
 

src/nextline_alert/plugin.py

@@ -5,6 +5,7 @@
 
 from apluggy import asynccontextmanager
 from dynaconf import Dynaconf, Validator
+
 from nextline import Nextline
 from nextlinegraphql.hook import spec
 

tests/test_emitter.py

@@ -4,8 +4,8 @@
 
 import pytest
 import respx
-from nextline import Nextline
 
+from nextline import Nextline
 from nextline_alert.emitter import Emitter