Use hatch fmt command (#1424)

Author: blink1073

.github/workflows/python-tests.yml

@@ -82,7 +82,7 @@ jobs:
       - name: Run Linters
         run: |
           hatch -v run typing:test
-          hatch -v run lint:build
+          hatch fmt
           pipx run interrogate -v .
           pipx run doc8 --max-line-length=200  --ignore-path=docs/source/other/full-config.rst
           npm install -g eslint

docs/source/conf.py

@@ -66,7 +66,7 @@
 
 # General information about the project.
 project = "Jupyter Server"
-copyright = "2020, Jupyter Team, https://jupyter.org"
+copyright = "2020, Jupyter Team, https://jupyter.org"  # noqa: A001
 author = "The Jupyter Team"
 
 # ghissue config

examples/simple/tests/test_handlers.py

@@ -3,7 +3,7 @@
 import pytest
 
 
-@pytest.fixture()
+@pytest.fixture
 def jp_server_auth_resources(jp_server_auth_core_resources):
     """The server auth resources."""
     for url_regex in [
@@ -13,7 +13,7 @@ def jp_server_auth_resources(jp_server_auth_core_resources):
     return jp_server_auth_core_resources
 
 
-@pytest.fixture()
+@pytest.fixture
 def jp_server_config(jp_template_dir, jp_server_authorizer):
     """The server config."""
     return {

jupyter_server/_sysinfo.py

@@ -42,7 +42,7 @@ def pkg_commit_hash(pkg_path):
         if p.exists(p.join(cur_path, ".git")):
             try:
                 proc = subprocess.Popen(
-                    ["git", "rev-parse", "--short", "HEAD"],
+                    ["git", "rev-parse", "--short", "HEAD"],  # noqa: S607
                     stdout=subprocess.PIPE,
                     stderr=subprocess.PIPE,
                     cwd=pkg_path,

jupyter_server/_tz.py

@@ -14,7 +14,7 @@
 ZERO = timedelta(0)
 
 
-class tzUTC(tzinfo):
+class tzUTC(tzinfo):  # noqa: N801
     """tzinfo object for UTC (zero offset)"""
 
     def utcoffset(self, d: datetime | None) -> timedelta:

jupyter_server/auth/authorizer.py

@@ -66,7 +66,7 @@ def is_authorized(
         bool
             True if user authorized to make request; False, otherwise
         """
-        raise NotImplementedError()
+        raise NotImplementedError
 
 
 class AllowAllAuthorizer(Authorizer):

jupyter_server/auth/security.py

@@ -68,13 +68,13 @@ def passwd(passphrase=None, algorithm="argon2"):
         )
         h_ph = ph.hash(passphrase)
 
-        return ":".join((algorithm, h_ph))
+        return f"{algorithm}:{h_ph}"
 
     h = hashlib.new(algorithm)
     salt = ("%0" + str(salt_len) + "x") % random.getrandbits(4 * salt_len)
     h.update(passphrase.encode("utf-8") + salt.encode("ascii"))
 
-    return ":".join((algorithm, salt, h.hexdigest()))
+    return f"{algorithm}:{salt}:{h.hexdigest()}"
 
 
 def passwd_check(hashed_passphrase, passphrase):

jupyter_server/auth/utils.py

@@ -166,4 +166,4 @@ def get_anonymous_username() -> str:
     Get a random user-name based on the moons of Jupyter.
     This function returns names like "Anonymous Io" or "Anonymous Metis".
     """
-    return moons_of_jupyter[random.randint(0, len(moons_of_jupyter) - 1)]
+    return moons_of_jupyter[random.randint(0, len(moons_of_jupyter) - 1)]  # noqa: S311

jupyter_server/base/call_context.py

@@ -44,7 +44,7 @@ def get(cls, name: str) -> Any:
 
         if name in name_value_map:
             return name_value_map[name]
-        return None  # TODO - should this raise `LookupError` (or a custom error derived from said)
+        return None  # TODO: should this raise `LookupError` (or a custom error derived from said)
 
     @classmethod
     def set(cls, name: str, value: Any) -> None:

jupyter_server/base/handlers.py

@@ -21,7 +21,6 @@
 import prometheus_client
 from jinja2 import TemplateNotFound
 from jupyter_core.paths import is_hidden
-from jupyter_events import EventLogger
 from tornado import web
 from tornado.log import app_log
 from traitlets.config import Application
@@ -45,6 +44,7 @@
 
 if TYPE_CHECKING:
     from jupyter_client.kernelspec import KernelSpecManager
+    from jupyter_events import EventLogger
     from jupyter_server_terminals.terminalmanager import TerminalManager
     from tornado.concurrent import Future
 
@@ -785,7 +785,7 @@ def get_login_url(self) -> str:
 
     @property
     def content_security_policy(self) -> str:
-        csp = "; ".join(
+        csp = "; ".join(  # noqa: FLY002
             [
                 super().content_security_policy,
                 "default-src 'none'",