review feedback

Author: FrankPortman

CHANGELOG.md

@@ -94,6 +94,7 @@ Unreleased changes template.
 * (toolchains) Run the check on the Python interpreter in isolated mode, to ensure it's not affected by userland environment variables, such as `PYTHONPATH`.
 * (toolchains) Ensure temporary `.pyc` and `.pyo` files are also excluded from the interpreters repository files.
 * (pypi) Run interpreter version call in isolated mode, to ensure it's not affected by userland environment variables, such as `PYTHONPATH`.
+* (packaging) An empty `requires_file` is treated as if it were omitted, resulting in a valid `METADATA` file.
 
 {#v0-0-0-added}
 ### Added

examples/wheel/wheel_test.py

@@ -483,7 +483,6 @@ def test_requires_file_and_extra_requires_files(self):
                     if line.startswith(b"Requires-Dist:"):
                         requires.append(line.decode("utf-8").strip())
 
-            print(requires)
             self.assertEqual(
                 [
                     "Requires-Dist: tomli>=2.0.0",
@@ -516,7 +515,6 @@ def test_empty_requires_file(self):
                 if line.startswith("Requires-Dist:"):
                     requires.append(line.strip())
 
-            print(requires)
             self.assertEqual([], requires)
 
     def test_minimal_data_files(self):

python/packaging.bzl

@@ -171,6 +171,11 @@ def py_wheel(
             These are subject to make var expansion, as with the `args` attribute.
             Note that you can also pass additional args to the bazel run command as in the example above.
         **kwargs: other named parameters passed to the underlying [py_wheel rule](#py_wheel_rule)
+
+    :::{versionchanged} VERSION_NEXT_FEATURE
+    From now on, an empty `requires_file` is treated as if it were omitted, resulting in a valid
+    `METADATA` file.
+    :::
     """
     tags = kwargs.pop("tags", [])
     manual_tags = depset(tags + ["manual"]).to_list()