pre-commit run ruff-format -a

Author: aignas

.agents/skills/buildkite-get-results/scripts/get_buildkite_results.py

@@ -85,7 +85,10 @@ def fetch_buildkite_data(build_url):
                     elif isinstance(jobs_data, dict) and "records" in jobs_data:
                         data["jobs"] = jobs_data["records"]
         except Exception as e:
-            print(f"Warning: Could not fetch detailed jobs from {jobs_url}: {e}", file=sys.stderr)
+            print(
+                f"Warning: Could not fetch detailed jobs from {jobs_url}: {e}",
+                file=sys.stderr,
+            )
 
     return data
 
@@ -165,15 +168,18 @@ def main():
 
     build_state = data.get("state", "Unknown")
     print(f"Build State: {build_state}")
-    
+
     jobs = data.get("jobs", [])
     jobs_count = data.get("statistics", {}).get("jobs_count", 0)
-    
+
     print(f"Total jobs reported: {jobs_count}")
     print(f"Jobs found in data: {len(jobs)}")
-    
+
     if jobs_count != len(jobs):
-        print(f"WARNING: Reported job count ({jobs_count}) does not match jobs found ({len(jobs)}).", file=sys.stderr)
+        print(
+            f"WARNING: Reported job count ({jobs_count}) does not match jobs found ({len(jobs)}).",
+            file=sys.stderr,
+        )
 
     print("-" * 40)
 

examples/bzlmod/entry_points/tests/pylint_deps_test.py

@@ -29,9 +29,9 @@ def __init__(self, *args, **kwargs):
 
     def test_pylint_entry_point(self):
         rlocation_path = os.environ.get("ENTRY_POINT")
-        assert (
-            rlocation_path is not None
-        ), "expected 'ENTRY_POINT' env variable to be set to rlocation of the tool"
+        assert rlocation_path is not None, (
+            "expected 'ENTRY_POINT' env variable to be set to rlocation of the tool"
+        )
 
         entry_point = pathlib.Path(runfiles.Create().Rlocation(rlocation_path))
         self.assertTrue(entry_point.exists(), f"'{entry_point}' does not exist")
@@ -55,9 +55,9 @@ def test_pylint_entry_point(self):
 
     def test_pylint_report_has_expected_warnings(self):
         rlocation_path = os.environ.get("PYLINT_REPORT")
-        assert (
-            rlocation_path is not None
-        ), "expected 'PYLINT_REPORT' env variable to be set to rlocation of the report"
+        assert rlocation_path is not None, (
+            "expected 'PYLINT_REPORT' env variable to be set to rlocation of the report"
+        )
 
         pylint_report = pathlib.Path(runfiles.Create().Rlocation(rlocation_path))
         self.assertTrue(pylint_report.exists(), f"'{pylint_report}' does not exist")

examples/bzlmod/entry_points/tests/pylint_test.py

@@ -28,9 +28,9 @@ def __init__(self, *args, **kwargs):
 
     def test_pylint_entry_point(self):
         rlocation_path = os.environ.get("ENTRY_POINT")
-        assert (
-            rlocation_path is not None
-        ), "expected 'ENTRY_POINT' env variable to be set to rlocation of the tool"
+        assert rlocation_path is not None, (
+            "expected 'ENTRY_POINT' env variable to be set to rlocation of the tool"
+        )
 
         entry_point = pathlib.Path(runfiles.Create().Rlocation(rlocation_path))
         self.assertTrue(entry_point.exists(), f"'{entry_point}' does not exist")

examples/bzlmod/entry_points/tests/yamllint_test.py

@@ -28,9 +28,9 @@ def __init__(self, *args, **kwargs):
 
     def test_yamllint_entry_point(self):
         rlocation_path = os.environ.get("ENTRY_POINT")
-        assert (
-            rlocation_path is not None
-        ), "expected 'ENTRY_POINT' env variable to be set to rlocation of the tool"
+        assert rlocation_path is not None, (
+            "expected 'ENTRY_POINT' env variable to be set to rlocation of the tool"
+        )
 
         entry_point = pathlib.Path(runfiles.Create().Rlocation(rlocation_path))
         self.assertTrue(entry_point.exists(), f"'{entry_point}' does not exist")

examples/bzlmod/tests/other_module/other_module_import_test.py

@@ -1,8 +1,10 @@
 """Regression test for https://github.com/bazel-contrib/rules_python/issues/3563"""
+
 import os
 import subprocess
 import sys
 
+
 def main():
     # The rlocation path for the bin_zipapp. It is in the "our_other_module" repository.
     zipapp_path = os.environ.get("ZIPAPP_PATH")
@@ -18,5 +20,6 @@ def main():
         print(f"bin_zippapp failed with return code {result.returncode}")
         sys.exit(result.returncode)
 
+
 if __name__ == "__main__":
     main()

examples/wheel/wheel_test.py

@@ -566,7 +566,9 @@ def test_extra_requires(self):
             )
 
     def test_requires_dist_depends_on_extras(self):
-        filename = self._get_path("requires_dist_depends_on_extras-0.0.1-py3-none-any.whl")
+        filename = self._get_path(
+            "requires_dist_depends_on_extras-0.0.1-py3-none-any.whl"
+        )
 
         with zipfile.ZipFile(filename) as zf:
             self.assertAllEntriesHasReproducibleMetadata(zf)
@@ -591,7 +593,9 @@ def test_requires_dist_depends_on_extras(self):
             )
 
     def test_requires_dist_depends_on_extras_file(self):
-        filename = self._get_path("requires_dist_depends_on_extras_using_file-0.0.1-py3-none-any.whl")
+        filename = self._get_path(
+            "requires_dist_depends_on_extras_using_file-0.0.1-py3-none-any.whl"
+        )
 
         with zipfile.ZipFile(filename) as zf:
             self.assertAllEntriesHasReproducibleMetadata(zf)

gazelle/manifest/copy_to_source.py

@@ -20,7 +20,9 @@ def copy_to_source(generated_relative_path: Path, target_relative_path: Path) ->
     generated_absolute_path = Path.cwd() / generated_relative_path
 
     # Similarly, the target is relative to the source directory.
-    target_absolute_path = os.environ["BUILD_WORKSPACE_DIRECTORY"] / target_relative_path
+    target_absolute_path = (
+        os.environ["BUILD_WORKSPACE_DIRECTORY"] / target_relative_path
+    )
 
     print(f"Copying {generated_absolute_path} to {target_absolute_path}")
     target_absolute_path.parent.mkdir(parents=True, exist_ok=True)

python/bin/repl_stub.py

@@ -62,10 +62,7 @@ def complete(self, text, state):
     elif "GNU readline" in readline.__doc__:  # type: ignore
         readline.parse_and_bind("tab: complete")
     else:
-        print(
-            "Could not enable tab completion: "
-            "unable to determine readline backend"
-        )
+        print("Could not enable tab completion: unable to determine readline backend")
 except ImportError:
     print(
         "Could not enable tab completion: "

python/private/get_local_runtime_info.py

@@ -48,9 +48,7 @@ def _search_directories(get_config, base_executable) -> list[str]:
     # On MacOS, the LDLIBRARY may be a relative path under /Library/Frameworks,
     # such as "Python.framework/Versions/3.12/Python", not a file under the
     # LIBDIR/LIBPL directory, so include PYTHONFRAMEWORKPREFIX.
-    lib_dirs = [
-        get_config(x) for x in ("PYTHONFRAMEWORKPREFIX", "LIBPL", "LIBDIR")
-    ]
+    lib_dirs = [get_config(x) for x in ("PYTHONFRAMEWORKPREFIX", "LIBPL", "LIBDIR")]
 
     # On Debian, with multiarch enabled, prior to Python 3.10, `LIBDIR` didn't
     # tell the location of the libs, just the base directory. The `MULTIARCH`
@@ -67,8 +65,8 @@ def _search_directories(get_config, base_executable) -> list[str]:
 
     if not _IS_DARWIN:
         for exec_dir in (
-                os.path.dirname(base_executable) if base_executable else None,
-                get_config("BINDIR"),
+            os.path.dirname(base_executable) if base_executable else None,
+            get_config("BINDIR"),
         ):
             if not exec_dir:
                 continue
@@ -122,7 +120,8 @@ def _search_library_names(get_config, version, abi_flags) -> list[str]:
     #
     # A typical LIBRARY is 'libpythonX.Y.a' on Linux.
     lib_names = [
-        get_config(x) for x in (
+        get_config(x)
+        for x in (
             "LDLIBRARY",
             "INSTSONAME",
             "PY3LIBRARY",
@@ -167,8 +166,7 @@ def _get_python_library_info(base_executable) -> dict[str, Any]:
     abi_flags = _get_abi_flags(config_vars.get)
 
     search_directories = _search_directories(config_vars.get, base_executable)
-    search_libnames = _search_library_names(config_vars.get, version,
-                                            abi_flags)
+    search_libnames = _search_library_names(config_vars.get, version, abi_flags)
 
     # Used to test whether the library is an abi3 library or a full api library.
     abi3_libraries = _default_library_names(sys.version_info.major, abi_flags)
@@ -221,10 +219,10 @@ def _get_python_library_info(base_executable) -> dict[str, Any]:
     # Additional DLLs are needed on Windows to link properly.
     dlls = []
     if _IS_WINDOWS:
-        dlls.extend(
-            glob.glob(os.path.join(os.path.dirname(base_executable), "*.dll")))
+        dlls.extend(glob.glob(os.path.join(os.path.dirname(base_executable), "*.dll")))
         dlls = [
-            x for x in dlls
+            x
+            for x in dlls
             if x not in dynamic_libraries and x not in abi_dynamic_libraries
         ]
 

python/private/pypi/dependency_resolver/dependency_resolver.py

@@ -159,7 +159,10 @@ def main(
         # Further, shutil.copy preserves the source file's mode, and so if
         # our source file is read-only (the default under Perforce Helix),
         # this scratch file will also be read-only, defeating its purpose.
-        with open(resolved_requirements_file, "rb") as fsrc, open(requirements_out, "wb") as fdst:
+        with (
+            open(resolved_requirements_file, "rb") as fsrc,
+            open(requirements_out, "wb") as fdst,
+        ):
             shutil.copyfileobj(fsrc, fdst)
 
     update_command = (