remove ai slop in comments

Author: RonnyPfannschmidt

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## Unreleased
 
+### Breaking
+
+- fix #1019: pass python version build tags from scm version to results propperly
+
 ### Added
 
 - add `setuptools-scm` console_scripts entry point to make the CLI directly executable

src/setuptools_scm/_overrides.py

@@ -164,7 +164,6 @@ def _read_pretended_version_for(
     pretended = read_named_env(name="PRETEND_VERSION", dist_name=config.dist_name)
 
     if pretended:
-        # Use enhanced meta function - let validation errors bubble up
         return version.meta(tag=pretended, preformatted=True, config=config)
     else:
         return None

src/setuptools_scm/fallbacks.py

@@ -25,7 +25,6 @@ def parse_pkginfo(root: _t.PathT, config: Configuration) -> ScmVersion | None:
     data = data_from_mime(pkginfo)
     version = data.get("Version", _UNKNOWN)
     if version != _UNKNOWN:
-        # Use enhanced meta function - let validation errors bubble up
         return meta(version, preformatted=True, config=config)
     else:
         return None
@@ -39,10 +38,8 @@ def fallback_version(root: _t.PathT, config: Configuration) -> ScmVersion | None
                 parent_name[len(config.parentdir_prefix_version) :], config
             )
             if version is not None:
-                # Use enhanced meta function - let validation errors bubble up
                 return meta(str(version), preformatted=True, config=config)
     if config.fallback_version is not None:
         log.debug("FALLBACK %s", config.fallback_version)
-        # Use enhanced meta function - let validation errors bubble up
         return meta(config.fallback_version, preformatted=True, config=config)
     return None

testing/test_cli.py

@@ -93,7 +93,6 @@ def test_cli_create_archival_file_stable(
     archival_file = wd.cwd / ".git_archival.txt"
     assert not archival_file.exists()
 
-    # Test successful creation
     result = main(["create-archival-file", "--stable"])
     assert result == 0
     assert archival_file.exists()
@@ -122,7 +121,6 @@ def test_cli_create_archival_file_full(
     archival_file = wd.cwd / ".git_archival.txt"
     assert not archival_file.exists()
 
-    # Test successful creation
     result = main(["create-archival-file", "--full"])
     assert result == 0
     assert archival_file.exists()

testing/test_integration.py

@@ -219,8 +219,7 @@ def test_pretend_metadata_with_version(
 
     # Test version file template functionality
     wd.write("setup.py", SETUP_PY_PLAIN)
-    wd("mkdir -p src")  # Create the src directory
-    # This is a template string, not an f-string - used by setuptools-scm templating
+    wd("mkdir -p src")
     version_file_content = """
 version = '{version}'
 major = {version_tuple[0]}
@@ -234,7 +233,6 @@ def test_pretend_metadata_with_version(
         write_to="src/version.py", write_to_template=version_file_content
     )
 
-    # Read the file using pathlib
     content = (wd.cwd / "src/version.py").read_text()
     assert "commit_hash = 'g1337beef'" in content
     assert "num_commit = 4" in content
@@ -266,10 +264,8 @@ def test_pretend_metadata_without_version_warns(
     # Let's create an empty git repo without commits to truly have no base version
     monkeypatch.setenv(PRETEND_METADATA_KEY, '{node="g1234567", distance=2}')
 
-    # Should get a version with fallback but metadata overrides applied
     with caplog.at_level(logging.WARNING):
         version = wd.get_version()
-        # Should get a fallback version with metadata overrides
         assert version is not None
 
     # In this case, metadata was applied to a fallback version, so no warning about missing base
@@ -301,7 +297,6 @@ def test_pretend_metadata_with_scm_version(
     # Test version file to see if metadata was applied
     wd.write("setup.py", SETUP_PY_PLAIN)
     wd("mkdir -p src")
-    # This is a template string, not an f-string - used by setuptools-scm templating
     version_file_content = """
 version = '{version}'
 commit_hash = '{scm_version.short_node}'
@@ -403,7 +398,6 @@ def test_git_tag_with_local_build_data_preserved(wd: WorkDir) -> None:
         f"Version should parse correctly as PEP 440: {version}"
     )
 
-    # Should preserve the build metadata that was in the git tag
     assert version == "1.0.0+build.123", (
         f"Expected build metadata preserved, got {version}"
     )
@@ -430,7 +424,6 @@ def test_git_tag_with_commit_hash_preserved(wd: WorkDir) -> None:
         f"Version should parse correctly as PEP 440: {version}"
     )
 
-    # Should preserve the commit hash that was in the git tag
     assert version == "2.0.0+sha.abcd1234"
 
     # Validate the local part is correct
@@ -459,7 +452,6 @@ def test_git_tag_with_local_build_data_preserved_dirty_workdir(wd: WorkDir) -> N
         f"Version should parse correctly as PEP 440: {version}"
     )
 
-    # Should preserve the build metadata that was in the git tag
     assert version == "1.5.0+build.456", (
         f"Expected build metadata preserved with dirty workdir, got {version}"
     )