precommit

Author: philsc

python/bin/BUILD.bazel

@@ -1,6 +1,6 @@
+load("//python:py_binary.bzl", "py_binary")
 load("//python/private:interpreter.bzl", _interpreter_binary = "interpreter_binary")
 load("//python/private:repl.bzl", "generate_repl_main")
-load("//python:py_binary.bzl", "py_binary")
 
 filegroup(
     name = "distribution",
@@ -27,8 +27,8 @@ label_flag(
 
 generate_repl_main(
     name = "repl_py",
-    out = "repl.py",
     src = "repl_stub.py",
+    out = "repl.py",
 )
 
 label_flag(
@@ -47,11 +47,11 @@ label_flag(
 py_binary(
     name = "repl",
     srcs = [":repl.py"],
+    visibility = ["//visibility:public"],
     deps = [
-        ":repl_stub_dep",
         ":repl_dep",
+        ":repl_stub_dep",
     ],
-    visibility = ["//visibility:public"],
 )
 
 # The user can modify this flag to make arbitrary PyInfo targets available for

python/bin/repl_stub.py

@@ -1,2 +1,3 @@
 import code
+
 code.interact(local=dict(globals(), **locals()))

python/private/repl.bzl

@@ -26,14 +26,14 @@ generate_repl_main = rule(
             mandatory = True,
             allow_single_file = True,
         ),
-        "_template": attr.label(
-            default = "//python/private:repl_template.py",
-            allow_single_file = True,
-        ),
         "_generator": attr.label(
             default = "//python/private:repl_main_generator",
             executable = True,
             cfg = "exec",
         ),
+        "_template": attr.label(
+            default = "//python/private:repl_template.py",
+            allow_single_file = True,
+        ),
     },
 )

python/private/repl_main_generator.py

@@ -3,9 +3,10 @@
 from pathlib import Path
 
 LINE_TO_REPLACE = """\
-    pass # %REPLACE_WHOLE_LINE_WITH_STUB%
+    pass  # %REPLACE_WHOLE_LINE_WITH_STUB%
 """
 
+
 def main(argv):
     template = Path(sys.argv[1])
     stub = Path(sys.argv[2])
@@ -18,7 +19,9 @@ def main(argv):
 
     output_text = template_text.replace(LINE_TO_REPLACE, indented_stub_text)
     if template_text == output_text:
-        raise ValueError("Failed to find the following in the template: {LINE_TO_REPLACE}")
+        raise ValueError(
+            f"Failed to find the following in the template: {LINE_TO_REPLACE}"
+        )
 
     output.write_text(output_text)
 

python/private/repl_template.py

@@ -1,14 +1,16 @@
 import os
 from pathlib import Path
 
+
 def repl_stub():
-    pass # %REPLACE_WHOLE_LINE_WITH_STUB%
+    pass  # %REPLACE_WHOLE_LINE_WITH_STUB%
+
 
 def start_repl():
     # Simulate Python's behavior when a valid startup script is defined by the
     # PYTHONSTARTUP variable. If this file path fails to load, print the error
     # and revert to the default behavior.
-    if (startup_file := os.getenv("PYTHONSTARTUP")):
+    if startup_file := os.getenv("PYTHONSTARTUP"):
         try:
             source_code = Path(startup_file).read_text()
         except Exception as error:
@@ -19,5 +21,6 @@ def start_repl():
 
     repl_stub()
 
+
 if __name__ == "__main__":
     start_repl()