Apply ruff rule RUF100 (#1401)

pypa · May 12, 2024 · 4b15fe2 · 4b15fe2
1 parent 964d93c
commit 4b15fe2
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 3 deletions.
diff --git a/changelog.d/1400.misc.md b/changelog.d/1400.misc.md
@@ -0,0 +1 @@
+Enable and apply ruff rule `RUF100`.
diff --git a/pyproject.toml b/pyproject.toml
@@ -66,6 +66,7 @@ lint.extend-select = [
   "PLE",
   "PLW",
   "RSE",
+  "RUF100",
   "W",
 ]
 lint.isort = {known-first-party = ["helpers", "package_info", "pipx"]}

diff --git a/src/pipx/__main__.py b/src/pipx/__main__.py
@@ -8,7 +8,7 @@
     pipx_package_source_path = os.path.dirname(os.path.dirname(__file__))
     sys.path.insert(0, pipx_package_source_path)
 
-from pipx.main import cli  # noqa
+from pipx.main import cli
 
 if __name__ == "__main__":
     sys.exit(cli())
diff --git a/src/pipx/interpreter.py b/src/pipx/interpreter.py
@@ -17,7 +17,7 @@
 
 def has_venv() -> bool:
     try:
-        import venv  # noqa
+        import venv  # noqa: F401
 
         return True
     except ImportError:

diff --git a/src/pipx/venv_inspect.py b/src/pipx/venv_inspect.py
@@ -108,7 +108,7 @@ def get_resources_from_inst_files(dist: metadata.Distribution, bin_path: Path, m
     # not sure what is found here
     inst_files = dist.read_text("installed-files.txt") or ""
     for line in inst_files.splitlines():
-        entry = line.split(",")[0]  # noqa: T484
+        entry = line.split(",")[0]
         inst_file_path = Path(str(dist.locate_file(entry))).resolve()
         try:
             if inst_file_path.parent.samefile(bin_path):