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

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

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.4.10 → v0.5.0](astral-sh/ruff-pre-commit@v0.4.10...v0.5.0)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* lint daemon.py

* lint run.py

* lint fixtures.py

* lint checkout.py

* lint collect.py

* lint index.py

* lint index.py

* lint checkout.py

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ivan Shcheklein <[email protected]>

.pre-commit-config.yaml

@@ -22,7 +22,7 @@ repos:
       - id: sort-simple-yaml
       - id: trailing-whitespace
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: 'v0.4.10'
+    rev: 'v0.5.0'
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]

dvc/daemon.py

@@ -44,10 +44,10 @@ def _win_detached_subprocess(args: Sequence[str], **kwargs) -> int:
 
     startupinfo = STARTUPINFO()
     startupinfo.dwFlags |= STARTF_USESHOWWINDOW
-    popen = subprocess.Popen(
+    popen = subprocess.Popen(  # noqa: S603
         args,
         close_fds=True,
-        shell=False,  # noqa: S603
+        shell=False,
         startupinfo=startupinfo,
         creationflags=creationflags,
         **kwargs,

dvc/repo/checkout.py

@@ -125,7 +125,7 @@ def onerror(target, exc):
             (StageFileDoesNotExistError, StageFileBadNameError, NoOutputOrStageError),
         ):
             raise CheckoutErrorSuggestGit(target) from exc
-        raise
+        raise  # noqa: PLE0704
 
     view = self.index.targets_view(
         targets, recursive=recursive, with_deps=with_deps, onerror=onerror

dvc/repo/experiments/collect.py

@@ -69,11 +69,11 @@ def collect_rev(
 
 def _collect_rev(
     repo: "Repo",
-    rev: str,
+    revision: str,
     param_deps: bool = False,
     **kwargs,
 ) -> SerializableExp:
-    with repo.switch(rev) as rev:
+    with repo.switch(revision) as rev:
         if rev == "workspace":
             timestamp: Optional[datetime] = None
         else:

dvc/repo/index.py

@@ -600,7 +600,7 @@ def collect_targets(
         if not onerror:
 
             def onerror(_target, _exc):
-                raise
+                raise  # noqa: PLE0704
 
         targets = ensure_list(targets)
         if not targets:

dvc/stage/run.py

@@ -27,8 +27,8 @@ def _fish_supports_no_config(executable) -> bool:
     bool: True if the fish version is greater than 3.3.0, False otherwise.
     """
     try:
-        output = subprocess.check_output(
-            [executable, "--version"],  # noqa: S603
+        output = subprocess.check_output(  # noqa: S603
+            [executable, "--version"],
             text=True,
         )
         version = Version(output.split(" ")[-1].strip())

dvc/testing/fixtures.py

@@ -228,10 +228,10 @@ def docker_services(tmp_path_factory, request):
         pytest.skip("disabled for Windows on CI")
 
     try:
-        subprocess.check_output(
+        subprocess.check_output(  # noqa: S602
             "docker ps",  # noqa: S607
             stderr=subprocess.STDOUT,
-            shell=True,  # noqa: S602
+            shell=True,
         )
     except subprocess.CalledProcessError as err:
         out = (err.output or b"").decode("utf-8")