lint

Author: philsc

python/bin/BUILD.bazel

@@ -9,11 +9,11 @@ filegroup(
 _interpreter_binary(
     name = "python",
     binary = ":python_src",
-    visibility = ["//visibility:public"],
     target_compatible_with = select({
         "@platforms//os:windows": ["@platforms//:incompatible"],
         "//conditions:default": [],
     }),
+    visibility = ["//visibility:public"],
 )
 
 # The user can modify this flag to source different interpreters for the

tests/interpreter/BUILD.bazel

@@ -12,22 +12,22 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-load(":interpreter_tests.bzl", "py_reconfig_interpreter_tests", "PYTHON_VERSIONS_TO_TEST")
+load(":interpreter_tests.bzl", "PYTHON_VERSIONS_TO_TEST", "py_reconfig_interpreter_tests")
 
 py_reconfig_interpreter_tests(
     name = "interpreter_version_test",
-    python_versions = PYTHON_VERSIONS_TO_TEST,
-    # Both the interpreter and the test itself are expected to run under
-    # the same version.
-    expected_interpreter_version = None,
     srcs = ["interpreter_test.py"],
     data = [
         "//python/bin:python",
     ],
     env = {
         "PYTHON_BIN": "$(rootpath //python/bin:python)",
     },
+    # Both the interpreter and the test itself are expected to run under
+    # the same version.
+    expected_interpreter_version = None,
     main = "interpreter_test.py",
+    python_versions = PYTHON_VERSIONS_TO_TEST,
 )
 
 py_reconfig_interpreter_tests(
@@ -39,11 +39,11 @@ py_reconfig_interpreter_tests(
     env = {
         "PYTHON_BIN": "$(rootpath //python/bin:python)",
     },
-    python_versions = PYTHON_VERSIONS_TO_TEST,
     # Since we're grabbing the interpreter from a binary with a fixed
     # version, we expect to always see that version. It doesn't matter what
     # Python version the test itself is running with.
     expected_interpreter_version = "3.11",
     main = "interpreter_test.py",
     python_src = "//tools/publish:twine",
+    python_versions = PYTHON_VERSIONS_TO_TEST,
 )

tests/interpreter/interpreter_tests.bzl

@@ -20,7 +20,7 @@ PYTHON_VERSIONS_TO_TEST = (
     "3.12",
 )
 
-def py_reconfig_interpreter_tests(name, python_versions, expected_interpreter_version=None, env={}, **kwargs):
+def py_reconfig_interpreter_tests(name, python_versions, expected_interpreter_version = None, env = {}, **kwargs):
     for python_version in python_versions:
         py_reconfig_test(
             name = "{}_{}".format(name, python_version),