Remove '+' on write mode (#1426)

Not required throughout the examples and docs

Signed-off-by: Elliot Gunton <elliotgunton@gmail.com>

Author: elliotgunton

docs/examples/workflows/artifacts/artifact_with_fanout.md

@@ -15,7 +15,7 @@
     def writer():
         import json
 
-        with open("/tmp/file", "w+") as f:
+        with open("/tmp/file", "w") as f:
             for i in range(10):
                 f.write(json.dumps(i) + "\n")
 
@@ -89,7 +89,7 @@
             import sys
             sys.path.append(os.getcwd())
             import json
-            with open('/tmp/file', 'w+') as f:
+            with open('/tmp/file', 'w') as f:
                 for i in range(10):
                     f.write(json.dumps(i) + '\n')
           command:

docs/examples/workflows/artifacts/basic_artifacts.md

@@ -15,7 +15,7 @@ task, consumer, takes this artifact, places it at its own `/file` path, and prin
 
     @script(outputs=Artifact(name="out-art", path="/tmp/file", archive=NoneArchiveStrategy()))
     def writer():
-        with open("/tmp/file", "w+") as f:
+        with open("/tmp/file", "w") as f:
             f.write("Hello, world!")
 
 
@@ -64,7 +64,7 @@ task, consumer, takes this artifact, places it at its own `/file` path, and prin
             import os
             import sys
             sys.path.append(os.getcwd())
-            with open('/tmp/file', 'w+') as f:
+            with open('/tmp/file', 'w') as f:
                 f.write('Hello, world!')
           command:
           - python

docs/walk-through/artifacts.md

@@ -78,7 +78,7 @@ from hera.workflows import Artifact, NoneArchiveStrategy, Steps, Workflow, scrip
 
 @script(outputs=Artifact(name="out-art", path="/tmp/file", archive=NoneArchiveStrategy()))
 def writer():
-    with open("/tmp/file", "w+") as f:
+    with open("/tmp/file", "w") as f:
         f.write("Hello, world!")
 
 
@@ -112,7 +112,7 @@ We then write to the specified path within the function:
 ```py
 @script(outputs=Artifact(name="out-art", path="/tmp/file", archive=NoneArchiveStrategy()))
 def writer():
-    with open("/tmp/file", "w+") as f:
+    with open("/tmp/file", "w") as f:
         f.write("Hello, world!")
 ```
 
@@ -159,7 +159,7 @@ We can then call the function in a Steps context. Compare to the YAML workflow a
             import os
             import sys
             sys.path.append(os.getcwd())
-            with open('/tmp/file', 'w+') as f:
+            with open('/tmp/file', 'w') as f:
                 f.write('Hello, world!')
           command:
           - python
@@ -242,7 +242,7 @@ output artifact to the `in-art` input artifact:
             import os
             import sys
             sys.path.append(os.getcwd())
-            with open('/tmp/file', 'w+') as f:
+            with open('/tmp/file', 'w') as f:
                 f.write('Hello, world!')
           command:
           - python

examples/workflows/artifacts/artifact-with-fanout.yaml

@@ -39,7 +39,7 @@ spec:
         import sys
         sys.path.append(os.getcwd())
         import json
-        with open('/tmp/file', 'w+') as f:
+        with open('/tmp/file', 'w') as f:
             for i in range(10):
                 f.write(json.dumps(i) + '\n')
       command:

examples/workflows/artifacts/artifact_with_fanout.py

@@ -5,7 +5,7 @@
 def writer():
     import json
 
-    with open("/tmp/file", "w+") as f:
+    with open("/tmp/file", "w") as f:
         for i in range(10):
             f.write(json.dumps(i) + "\n")
 

examples/workflows/artifacts/basic-artifacts.yaml

@@ -28,7 +28,7 @@ spec:
         import os
         import sys
         sys.path.append(os.getcwd())
-        with open('/tmp/file', 'w+') as f:
+        with open('/tmp/file', 'w') as f:
             f.write('Hello, world!')
       command:
       - python

examples/workflows/artifacts/basic_artifacts.py

@@ -8,7 +8,7 @@
 
 @script(outputs=Artifact(name="out-art", path="/tmp/file", archive=NoneArchiveStrategy()))
 def writer():
-    with open("/tmp/file", "w+") as f:
+    with open("/tmp/file", "w") as f:
         f.write("Hello, world!")
 
 

scripts/models.py

@@ -119,7 +119,7 @@ def get_import_paths_from_refs(refs: list, root_path: str) -> List[ImportReferen
 def write_imports(references: List[ImportReference], models_type: str, openapi_spec_url: str) -> None:
     """Writes the given imports to the `root_path` models."""
     path = Path(__name__).parent.parent / "src" / "hera" / models_type / "models" / "__init__.py"
-    with open(str(path), "w+") as f:
+    with open(str(path), "w") as f:
         f.write(
             f'"""[DO NOT EDIT MANUALLY] Auto-generated model classes.\n\n'
             f"Auto-generated by Hera via `make {models_type}-models`.\n"

scripts/service.py

@@ -503,7 +503,7 @@ def make_service(service_def: str, endpoints: List[ServiceEndpoint], models_type
 
 def write_service(service: str, path: Path) -> None:
     """Writes the service code to the specified path."""
-    with open(str(path), "w+") as f:
+    with open(str(path), "w") as f:
         f.write(service)
 
 

scripts/spec.py

@@ -165,5 +165,5 @@
 
 # finally, we write the spec to the output file that is passed to use assuming the client wants to perform
 # something with this file
-with open(output_file, "w+") as f:
+with open(output_file, "w") as f:
     json.dump(spec, f, indent=2)