[pre-commit.ci] pre-commit autoupdate (#389)

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/python-jsonschema/check-jsonschema: 0.21.0 → 0.22.0](python-jsonschema/check-jsonschema@0.21.0...0.22.0)
- [github.com/psf/black: 23.1.0 → 23.3.0](psf/black@23.1.0...23.3.0)
- [github.com/charliermarsh/ruff-pre-commit: v0.0.254 → v0.0.260](astral-sh/ruff-pre-commit@v0.0.254...v0.0.260)

* Lint code & align versions

* Missed linter error

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Frédéric Collonval <[email protected]>

Author: pre-commit-ci[bot]

.pre-commit-config.yaml

@@ -19,7 +19,7 @@ repos:
       - id: trailing-whitespace
 
   - repo: https://github.com/python-jsonschema/check-jsonschema
-    rev: 0.21.0
+    rev: 0.22.0
     hooks:
       - id: check-github-workflows
 
@@ -31,12 +31,12 @@ repos:
           [mdformat-gfm, mdformat-frontmatter, mdformat-footnote]
 
   - repo: https://github.com/psf/black
-    rev: 23.1.0
+    rev: 23.3.0
     hooks:
       - id: black
 
   - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: v0.0.254
+    rev: v0.0.260
     hooks:
       - id: ruff
         args: ["--fix"]

jupyterlab_server/handlers.py

@@ -263,7 +263,7 @@ def add_handlers(handlers, extension_app):  # noqa
     allowed_extensions_uris = settings_config.get("allowed_extensions_uris", "")
 
     if (blocked_extensions_uris) and (allowed_extensions_uris):
-        warnings.warn(
+        warnings.warn(  # noqa B028
             "Simultaneous blocked_extensions_uris and allowed_extensions_uris is not supported. Please define only one of those."
         )
         import sys

jupyterlab_server/translations_handler.py

@@ -52,7 +52,7 @@ async def get(self, locale=None):
                 data, message = await current_loop.run_in_executor(
                     None, partial(get_language_pack, locale)
                 )
-                if data == {} and message == "":
+                if data == {} and not message:
                     if is_valid_locale(locale):
                         message = f"Language pack '{locale}' not installed!"
                     else:

jupyterlab_server/workspaces_app.py

@@ -180,7 +180,7 @@ def _validate(self, data):
 
         # If workspace_name is set in config, inject the
         # name into the workspace metadata.
-        if self.workspace_name is not None and self.workspace_name != "":
+        if self.workspace_name is not None and self.workspace_name:
             workspace["metadata"] = {"id": self.workspace_name}
         elif "id" not in workspace["metadata"]:
             msg = "The `id` field is missing in `metadata`."

pyproject.toml

@@ -116,10 +116,10 @@ test = "mypy --install-types --non-interactive {args:jupyterlab_server tests}"
 
 [tool.hatch.envs.lint]
 dependencies = [
-  "black[jupyter]==23.1.0",
+  "black[jupyter]==23.3.0",
   "mdformat>0.7",
   "mdformat-gfm>=0.3.5",
-  "ruff==0.0.254"
+  "ruff==0.0.260"
 ]
 detatched = true
 [tool.hatch.envs.lint.scripts]
@@ -249,8 +249,14 @@ ignore = [
 # F841 Local variable `list_data` is assigned to but never used
 # EM101 Exception must not use a string literal
 "tests/*" = ["S101", "A001", "F841", "EM101", "EM102", "EM103", "PLR2004"]
+# T201 `print` found
+"jupyterlab_server/licenses_app.py" = ["T201"]
+# T201 `print` found
+"jupyterlab_server/process.py" = ["T201"]
 # F401 `foo` imported but unused
 "jupyterlab_server/server.py" = ["F401"]
+# T201 `print` found
+"jupyterlab_server/workspaces_app.py" = ["B028", "T201"]
 # Invalid module name
 "tests/translations/jupyterlab-language-pack-es_CO/jupyterlab_language_pack_es_CO/__init__.py" = ["N999"]
 

tests/test_labapp.py

@@ -52,7 +52,7 @@ async def test_page_config(labserverapp, jp_fetch):
     # Check that the lab template is loaded
     html = r.body.decode()
     page_config = extract_page_config(html)
-    assert page_config['treePath'] == ""
+    assert not page_config['treePath']
     assert page_config['preferredPath'] == "/"
 
     def ispath(p):

tests/test_process.py

@@ -46,4 +46,4 @@ class TestApp(ProcessApp):
     # Kandle exception on older versions of server.
     except Exception as e:
         # Convert to warning so the test will pass on min version test.
-        warnings.warn(str(e))
+        warnings.warn(str(e))  # noqa B028

tests/test_translation_api.py

@@ -143,28 +143,28 @@ async def test_backend_locale_extension(jp_fetch):
 def test_get_installed_language_pack_locales_passes():
     data, message = _get_installed_language_pack_locales()
     assert "es_CO" in data
-    assert message == ""
+    assert not message
 
 
 def test_get_installed_package_locales():
     data, message = _get_installed_package_locales()
     assert "jupyterlab_some_package" in data
     assert os.path.isdir(data["jupyterlab_some_package"])
-    assert message == ""
+    assert not message
 
 
 def test_get_installed_packages_locale():
     data, message = get_installed_packages_locale("es_CO")
     assert "jupyterlab_some_package" in data
     assert "" in data["jupyterlab_some_package"]
-    assert message == ""
+    assert not message
 
 
 def test_get_language_packs():
     data, message = get_language_packs("en")
     assert "en" in data
     assert "es_CO" in data
-    assert message == ""
+    assert not message
 
 
 def test_get_language_pack():
@@ -173,7 +173,7 @@ def test_get_language_pack():
     assert "jupyterlab_some_package" in data
     assert "" in data["jupyterlab"]
     assert "" in data["jupyterlab_some_package"]
-    assert message == ""
+    assert not message
 
 
 # --- Utils