Fix Ruff file exclusion

backend/src/hatchling/cli/build/__init__.py

@@ -86,7 +86,7 @@ def build_impl(
                 app.display_info(artifact)
 
 
-def build_command(subparsers: argparse._SubParsersAction, defaults: Any) -> None:
+def build_command(subparsers: argparse._SubParsersAction, defaults: Any) -> None:  # noqa: SLF001
     parser = subparsers.add_parser('build')
     parser.add_argument(
         '-d', '--directory', dest='directory', help='The directory in which to build artifacts', **defaults

ruff.toml

@@ -1,5 +1,8 @@
 extend = "ruff_defaults.toml"
 
+# https://github.com/astral-sh/ruff/issues/8627
+exclude = [".git", ".mypy_cache", ".ruff_cache", ".venv", "dist"]
+
 [lint]
 preview = true
 ignore = [

src/hatch/cli/build/__init__.py

@@ -54,17 +54,13 @@ def build(app: Application, location, targets, hooks_only, no_hooks, ext, clean,
     """Build a project."""
     app.ensure_environment_plugin_dependencies()
 
-    from hatchling.builders.constants import BuildEnvVars
-    from hatchling.builders.plugin.interface import BuilderInterface
-
     from hatch.config.constants import AppEnvVars
     from hatch.utils.fs import Path
     from hatch.utils.structures import EnvVars
+    from hatchling.builders.constants import BuildEnvVars
+    from hatchling.builders.plugin.interface import BuilderInterface
 
-    if location:
-        path = str(Path(location).resolve())
-    else:
-        path = None
+    path = str(Path(location).resolve()) if location else None
 
     if ext:
         hooks_only = True
@@ -93,14 +89,14 @@ def build(app: Application, location, targets, hooks_only, no_hooks, ext, clean,
         env_vars[AppEnvVars.QUIET] = str(abs(app.verbosity))
 
     class Builder(BuilderInterface):
-        def get_version_api(self):
+        def get_version_api(self):  # noqa: PLR6301
             return {}
 
     with app.project.location.as_cwd(env_vars):
         environment = app.get_environment()
         try:
             environment.check_compatibility()
-        except Exception as e:
+        except Exception as e:  # noqa: BLE001
             app.abort(f'Environment `{environment.name}` is incompatible: {e}')
 
         for target in targets:

src/hatch/venv/core.py

@@ -42,7 +42,7 @@ def deactivate(self):
 
         self._env_vars_to_restore.clear()
 
-    def create(self, python, allow_system_packages=False):
+    def create(self, python, *, allow_system_packages=False):
         # WARNING: extremely slow import
         from virtualenv import cli_run
 
@@ -76,16 +76,19 @@ def executables_directory(self):
                 if exe_dir.is_dir():
                     self._executables_directory = exe_dir
                 else:
-                    raise OSError(f'Unable to locate executables directory within: {self.directory}')
+                    msg = f'Unable to locate executables directory within: {self.directory}'
+                    raise OSError(msg)
             # Debian
             elif (self.directory / 'local').is_dir():  # no cov
                 exe_dir = self.directory / 'local' / 'bin'
                 if exe_dir.is_dir():
                     self._executables_directory = exe_dir
                 else:
-                    raise OSError(f'Unable to locate executables directory within: {self.directory}')
+                    msg = f'Unable to locate executables directory within: {self.directory}'
+                    raise OSError(msg)
             else:
-                raise OSError(f'Unable to locate executables directory within: {self.directory}')
+                msg = f'Unable to locate executables directory within: {self.directory}'
+                raise OSError(msg)
 
         return self._executables_directory
 

tests/cli/build/test_build.py

@@ -80,20 +80,20 @@ def test_legacy(self, hatch, temp_dir, helpers):
         data_path.mkdir()
 
         (path / 'pyproject.toml').write_text(
-                """\
+            """\
 [build-system]
 requires = ["setuptools"]
 build-backend = "setuptools.build_meta"
 """
         )
         (path / 'setup.py').write_text(
-                """\
+            """\
 import setuptools
 setuptools.setup(name="tmp", version="0.0.1")
 """
         )
         (path / 'tmp.py').write_text(
-                """\
+            """\
 print("Hello World!")
 """
         )
@@ -208,8 +208,8 @@ def test_default(hatch, temp_dir, helpers):
     artifacts = list(build_directory.iterdir())
     assert len(artifacts) == 2
 
-    sdist_path = [artifact for artifact in artifacts if artifact.name.endswith('.tar.gz')][0]
-    wheel_path = [artifact for artifact in artifacts if artifact.name.endswith('.whl')][0]
+    sdist_path = next(artifact for artifact in artifacts if artifact.name.endswith('.tar.gz'))
+    wheel_path = next(artifact for artifact in artifacts if artifact.name.endswith('.whl'))
 
     assert result.output == helpers.dedent(
         f"""
@@ -243,7 +243,7 @@ def test_explicit_targets(hatch, temp_dir, helpers):
     artifacts = list(build_directory.iterdir())
     assert len(artifacts) == 1
 
-    wheel_path = [artifact for artifact in artifacts if artifact.name.endswith('.whl')][0]
+    wheel_path = next(artifact for artifact in artifacts if artifact.name.endswith('.whl'))
 
     assert result.output == helpers.dedent(
         f"""
@@ -273,8 +273,8 @@ def test_explicit_directory(hatch, temp_dir, helpers):
     artifacts = list(build_directory.iterdir())
     assert len(artifacts) == 2
 
-    sdist_path = [artifact for artifact in artifacts if artifact.name.endswith('.tar.gz')][0]
-    wheel_path = [artifact for artifact in artifacts if artifact.name.endswith('.whl')][0]
+    sdist_path = next(artifact for artifact in artifacts if artifact.name.endswith('.tar.gz'))
+    wheel_path = next(artifact for artifact in artifacts if artifact.name.endswith('.whl'))
 
     assert result.output == helpers.dedent(
         f"""
@@ -307,8 +307,8 @@ def test_explicit_directory_env_var(hatch, temp_dir, helpers):
     artifacts = list(build_directory.iterdir())
     assert len(artifacts) == 2
 
-    sdist_path = [artifact for artifact in artifacts if artifact.name.endswith('.tar.gz')][0]
-    wheel_path = [artifact for artifact in artifacts if artifact.name.endswith('.whl')][0]
+    sdist_path = next(artifact for artifact in artifacts if artifact.name.endswith('.tar.gz'))
+    wheel_path = next(artifact for artifact in artifacts if artifact.name.endswith('.whl'))
 
     assert result.output == helpers.dedent(
         f"""
@@ -389,10 +389,10 @@ def initialize(self, version, build_data):
     assert len(artifacts) == 3
     assert test_file in artifacts
 
-    sdist_path = [artifact for artifact in artifacts if artifact.name.endswith('.tar.gz')][0]
+    sdist_path = next(artifact for artifact in artifacts if artifact.name.endswith('.tar.gz'))
     assert '9000' in str(sdist_path)
 
-    wheel_path = [artifact for artifact in artifacts if artifact.name.endswith('.whl')][0]
+    wheel_path = next(artifact for artifact in artifacts if artifact.name.endswith('.whl'))
     assert '9000' in str(wheel_path)
 
     assert result.output == helpers.dedent(
@@ -446,10 +446,10 @@ def test_clean_env_var(hatch, temp_dir, helpers):
     assert len(artifacts) == 3
     assert test_file in artifacts
 
-    sdist_path = [artifact for artifact in artifacts if artifact.name.endswith('.tar.gz')][0]
+    sdist_path = next(artifact for artifact in artifacts if artifact.name.endswith('.tar.gz'))
     assert '9000' in str(sdist_path)
 
-    wheel_path = [artifact for artifact in artifacts if artifact.name.endswith('.whl')][0]
+    wheel_path = next(artifact for artifact in artifacts if artifact.name.endswith('.whl'))
     assert '9000' in str(wheel_path)
 
     assert result.output == helpers.dedent(
@@ -646,8 +646,8 @@ def initialize(self, version, build_data):
     artifacts = list(build_directory.iterdir())
     assert len(artifacts) == 2
 
-    sdist_path = [artifact for artifact in artifacts if artifact.name.endswith('.tar.gz')][0]
-    wheel_path = [artifact for artifact in artifacts if artifact.name.endswith('.whl')][0]
+    sdist_path = next(artifact for artifact in artifacts if artifact.name.endswith('.tar.gz'))
+    wheel_path = next(artifact for artifact in artifacts if artifact.name.endswith('.whl'))
 
     assert result.output == helpers.dedent(
         f"""
@@ -709,8 +709,8 @@ def initialize(self, version, build_data):
     artifacts = list(build_directory.iterdir())
     assert len(artifacts) == 2
 
-    sdist_path = [artifact for artifact in artifacts if artifact.name.endswith('.tar.gz')][0]
-    wheel_path = [artifact for artifact in artifacts if artifact.name.endswith('.whl')][0]
+    sdist_path = next(artifact for artifact in artifacts if artifact.name.endswith('.tar.gz'))
+    wheel_path = next(artifact for artifact in artifacts if artifact.name.endswith('.whl'))
 
     assert result.output == helpers.dedent(
         f"""
@@ -998,7 +998,7 @@ def initialize(self, version, build_data):
     assert len(artifacts) == 1
     assert not (path / 'my_app' / 'lib.so').exists()
 
-    wheel_path = [artifact for artifact in artifacts if artifact.name.endswith('.whl')][0]
+    wheel_path = next(artifact for artifact in artifacts if artifact.name.endswith('.whl'))
 
     assert result.output == helpers.dedent(
         f"""
@@ -1051,7 +1051,7 @@ def initialize(self, version, build_data):
     assert len(artifacts) == 1
     assert not (path / 'my_app' / 'lib.so').exists()
 
-    wheel_path = [artifact for artifact in artifacts if artifact.name.endswith('.whl')][0]
+    wheel_path = next(artifact for artifact in artifacts if artifact.name.endswith('.whl'))
 
     assert result.output == helpers.dedent(
         f"""
@@ -1082,7 +1082,7 @@ def test_debug_verbosity(hatch, temp_dir, helpers):
     artifacts = list(build_directory.iterdir())
     assert len(artifacts) == 1
 
-    wheel_path = [artifact for artifact in artifacts if artifact.name.endswith('.whl')][0]
+    wheel_path = next(artifact for artifact in artifacts if artifact.name.endswith('.whl'))
 
     assert result.output == helpers.dedent(
         f"""
@@ -1253,8 +1253,8 @@ def test_plugin_dependencies_unmet(hatch, temp_dir, helpers, mock_plugin_install
     artifacts = list(build_directory.iterdir())
     assert len(artifacts) == 2
 
-    sdist_path = [artifact for artifact in artifacts if artifact.name.endswith('.tar.gz')][0]
-    wheel_path = [artifact for artifact in artifacts if artifact.name.endswith('.whl')][0]
+    sdist_path = next(artifact for artifact in artifacts if artifact.name.endswith('.tar.gz'))
+    wheel_path = next(artifact for artifact in artifacts if artifact.name.endswith('.whl'))
 
     assert result.output == helpers.dedent(
         f"""

tests/venv/test_core.py

@@ -44,7 +44,8 @@ def test_executables_directory(temp_dir, platform):
         if entry.name.startswith('py'):
             break
     else:  # no cov
-        raise AssertionError('Unable to locate Python executable')
+        msg = 'Unable to locate Python executable'
+        raise AssertionError(msg)
 
 
 def test_activation(temp_dir, platform):