Backport PR #15499: Adopt ruff format (#15564)

(cherry picked from commit 5ec3aa4)

Co-authored-by: Jeremy Tuloup <[email protected]>

Author: krassowski

.pre-commit-config.yaml

@@ -31,17 +31,13 @@ repos:
     hooks:
       - id: check-github-workflows
 
-  - repo: https://github.com/psf/black
-    rev: 23.10.1
-    hooks:
-      - id: black
-
   # Check ruff version is aligned with the one in pyproject.toml
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.1.4
+    rev: v0.1.6
     hooks:
       - id: ruff
         args: ["--fix"]
+      - id: ruff-format
 
   - repo: local
     hooks:

binder/environment.yml

@@ -39,7 +39,6 @@ dependencies:
   - requests-cache
   - virtualenv
   # dev
-  - black-jupyter
   - bump2version
   - hatch
   - pre-commit

clean.py

@@ -12,7 +12,9 @@
     for root, dnames, _ in os.walk(here):
         if "node_modules" in dnames:
             subprocess.check_call(
-                ["rmdir", "/s", "/q", "node_modules"], cwd=root, shell=True  # noqa S602 S607
+                ["rmdir", "/s", "/q", "node_modules"],  # noqa S602 S607
+                cwd=root,
+                shell=True,  # noqa S602 S607
             )
             dnames.remove("node_modules")
 

docs/source/conf.py

@@ -394,7 +394,7 @@ def document_plugins_tokens_list(list_path: Path, output_path: Path) -> None:
 
 
 # Example configuration for intersphinx: refer to the Python standard library.
-intersphinx_mapping = {'python': ('https://docs.python.org/3', None)}
+intersphinx_mapping = {"python": ("https://docs.python.org/3", None)}
 
 
 def setup(app):

docs/source/developer/contributing.rst

@@ -97,11 +97,11 @@ All source code is written in
 `TypeScript <https://www.typescriptlang.org/Handbook>`__. See the `Style
 Guide <https://github.com/jupyterlab/jupyterlab/wiki/TypeScript-Style-Guide>`__.
 
-All non-python source code is formatted using `prettier <https://prettier.io>`__, and python source code is formatted using `black <https://github.com/psf/black>`__.
+All non-python source code is formatted using `prettier <https://prettier.io>`__, and python source code is formatted using `ruff <https://docs.astral.sh/ruff>`__.
 When code is modified and committed, all staged files will be
 automatically formatted using pre-commit git hooks (with help from
 `pre-commit <https://github.com/pre-commit/pre-commit>`__). The benefit of
-using a code formatters like ``prettier`` and ``black`` is that it removes the topic of
+using a code formatters like ``prettier`` and ``ruff`` is that it removes the topic of
 code style from the conversation when reviewing pull requests, thereby
 speeding up the review process.
 
@@ -121,7 +121,7 @@ You can invoke the pre-commit hook by hand at any time with::
 
 which should run any autoformatting on your code
 and tell you about any errors it couldn't fix automatically.
-You may also install `black integration <https://github.com/psf/black#editor-integration>`__
+You may also install `ruff integration <https://docs.astral.sh/ruff/integrations>`__
 into your text editor to format code automatically.
 
 If you have already committed files before setting up the pre-commit

examples/example_check.py

@@ -21,7 +21,7 @@
 
 here = Path(__file__).parent.resolve()
 TEST_FILE = here / "example.spec.ts"
-REF_SNAPSHOT = Path(TEST_FILE.with_suffix('.ts-snapshots').name) / "example-linux.png"
+REF_SNAPSHOT = Path(TEST_FILE.with_suffix(".ts-snapshots").name) / "example-linux.png"
 
 
 def main():
@@ -96,7 +96,7 @@ async def run_browser(url):
 
     current_env = os.environ.copy()
     current_env["BASE_URL"] = url
-    current_env["TEST_SNAPSHOT"] = '1' if has_snapshot else '0'
+    current_env["TEST_SNAPSHOT"] = "1" if has_snapshot else "0"
     try:
         await run_async_process(["npx", "playwright", "test"], env=current_env, cwd=str(target))
     finally:

jupyterlab/commands.py

@@ -1994,7 +1994,9 @@ def _yarn_config(logger):
 
     try:
         output_binary = subprocess.check_output(
-            [node, YARN_PATH, "config", "--json"], stderr=subprocess.PIPE, cwd=HERE  # noqa S603
+            [node, YARN_PATH, "config", "--json"],  # noqa S603
+            stderr=subprocess.PIPE,
+            cwd=HERE,
         )
         output = output_binary.decode("utf-8")
         lines = iter(output.splitlines())
@@ -2328,7 +2330,7 @@ def _log_multiple_compat_errors(logger, errors_map):
                 [
                     "\n   The following extensions are outdated:",
                     *outdated,
-                    '\n   Consider checking if an update is available for these packages.\n',
+                    "\n   Consider checking if an update is available for these packages.\n",
                 ]
             )
         )

jupyterlab/federated_labextensions.py

@@ -442,7 +442,8 @@ def _get_labextension_metadata(module):  # noqa
         try:
             package = (
                 subprocess.check_output(
-                    [sys.executable, "setup.py", "--name"], cwd=mod_path  # noqa S603
+                    [sys.executable, "setup.py", "--name"],  # noqa S603
+                    cwd=mod_path,
                 )
                 .decode("utf8")
                 .strip()

jupyterlab/handlers/announcements.py

@@ -238,18 +238,18 @@ def get_xml_text(attr: str, default: Optional[str] = None) -> str:
                             return default
                         else:
                             error_m = (
-                                f'atom feed entry does not contain a required attribute: {attr}'
+                                f"atom feed entry does not contain a required attribute: {attr}"
                             )
                             raise KeyError(error_m)
 
                     entry_title = get_xml_text("title")
                     entry_id = get_xml_text("id")
                     entry_updated = get_xml_text("updated")
                     entry_published = get_xml_text("published", entry_updated)
-                    entry_summary = get_xml_text("summary", default='')
+                    entry_summary = get_xml_text("summary", default="")
                     links = node.findall("atom:link", xml_namespaces)
                     if len(links) > 1:
-                        alternate = list(filter(lambda elem: elem.get('rel') == 'alternate', links))
+                        alternate = list(filter(lambda elem: elem.get("rel") == "alternate", links))
                         link_node = alternate[0] if alternate else links[0]
                     else:
                         link_node = links[0] if len(links) == 1 else None

jupyterlab/tests/test_extensions.py

@@ -239,7 +239,6 @@ async def mock_pkg_metadata(n, l, b):  # noqa
                     "nbformat",
                     "packaging",
                     "pexpect",
-                    "black ; extra == 'dev'",
                     "coverage ; extra == 'dev'",
                     "jupyter-packaging (~=0.7.9) ; extra == 'dev'",
                     "jupyterlab (~=3.0) ; extra == 'dev'",
@@ -248,7 +247,6 @@ async def mock_pkg_metadata(n, l, b):  # noqa
                     "pytest-asyncio ; extra == 'dev'",
                     "pytest-cov ; extra == 'dev'",
                     "pytest-tornasync ; extra == 'dev'",
-                    "black ; extra == 'tests'",
                     "coverage ; extra == 'tests'",
                     "jupyter-packaging (~=0.7.9) ; extra == 'tests'",
                     "jupyterlab (~=3.0) ; extra == 'tests'",