Merge pull request #107 from timkpaine/tkp/ruff

update ruff usage

Author: timkpaine

Makefile

@@ -27,18 +27,18 @@ tests: testpy testjs ## run the tests
 ###########
 # Linting #
 ###########
-lintpy:  ## Black/flake8 python
-	python -m ruff jupyterlab_commands setup.py
-	python -m black --check jupyterlab_commands setup.py
+lintpy:  ## lint python with ruff
+	python -m ruff check jupyterlab_commands
+	python -m ruff format --check jupyterlab_commands
 
 lintjs:  ## ESlint javascript
 	cd js; yarn lint
 
 lint: lintpy lintjs  ## run linter
 
-fixpy:  ## Black python
-	python -m ruff jupyterlab_commands setup.py --fix
-	python -m black jupyterlab_commands/ setup.py
+fixpy:  ## autoformat python with ruff
+	python -m ruff check --fix jupyterlab_commands
+	python -m ruff format jupyterlab_commands
 
 fixjs:  ## ESlint Autofix JS
 	cd js; yarn fix

jupyterlab_commands/extension.py

@@ -49,10 +49,7 @@ def load_jupyter_server_extension(nb_server_app):
     base_url = web_app.settings["base_url"]
 
     host_pattern = ".*$"
-    nb_server_app.log.info(
-        "Installing jupyterlab_commands handler on path %s"
-        % url_path_join(base_url, "commands")
-    )
+    nb_server_app.log.info("Installing jupyterlab_commands handler on path %s" % url_path_join(base_url, "commands"))
 
     nb_server_app.log.info("Available commands: %s" % ",".join(k for k in commands))
     web_app.add_handlers(

jupyterlab_commands/tests/test_init.py

@@ -3,6 +3,4 @@
 
 class TestInit:
     def test__jupyter_server_extension_paths(self):
-        assert _jupyter_server_extension_paths() == [
-            {"module": "jupyterlab_commands.extension"}
-        ]
+        assert _jupyter_server_extension_paths() == [{"module": "jupyterlab_commands.extension"}]

pyproject.toml

@@ -38,7 +38,6 @@ dependencies = [
 
 [project.optional-dependencies]
 develop = [
-    "black>=23",
     "check-manifest",
     "ruff",
     "pytest",
@@ -96,5 +95,17 @@ build_cmd = "build"
 [tool.ruff]
 line-length = 120
 
-[tool.ruff.per-file-ignores]
-"__init__.py" = ["F401"]
+[tool.ruff.lint.isort]
+combine-as-imports = true
+default-section = "third-party"
+known-first-party = ["jupyterlab_commands"]
+section-order = [
+    "future",
+    "standard-library",
+    "third-party",
+    "first-party",
+    "local-folder",
+]
+
+[tool.ruff.lint.per-file-ignores]
+"__init__.py" = ["F401", "F403"]