Remove unrelated security check, will make own PR

Author: hugovk

release.py

@@ -68,11 +68,6 @@ def get(
     @overload
     def get(self, key: Literal["sign_gpg"], default: bool | None = None) -> bool: ...
 
-    @overload
-    def get(
-        self, key: Literal["security_release"], default: bool | None = None
-    ) -> bool: ...
-
     @overload
     def get(self, key: Literal["release"], default: Tag | None = None) -> Tag: ...
 
@@ -100,9 +95,6 @@ def __getitem__(self, key: Literal["ssh_key"]) -> str | None: ...
     @overload
     def __getitem__(self, key: Literal["sign_gpg"]) -> bool: ...
 
-    @overload
-    def __getitem__(self, key: Literal["security_release"]) -> bool: ...
-
     @overload
     def __getitem__(self, key: Literal["release"]) -> Tag: ...
 
@@ -132,9 +124,6 @@ def __setitem__(self, key: Literal["ssh_key"], value: str | None) -> None: ...
     @overload
     def __setitem__(self, key: Literal["sign_gpg"], value: bool) -> None: ...
 
-    @overload
-    def __setitem__(self, key: Literal["security_release"], value: bool) -> None: ...
-
     @overload
     def __setitem__(self, key: Literal["release"], value: Tag) -> None: ...
 

run_release.py

@@ -222,7 +222,6 @@ def __init__(
         ssh_user: str,
         sign_gpg: bool,
         ssh_key: str | None = None,
-        security_release: bool = False,
         first_state: Task | None = None,
     ) -> None:
         self.tasks = tasks
@@ -249,9 +248,6 @@ def __init__(
             self.db["ssh_key"] = ssh_key
         if not self.db.get("sign_gpg"):
             self.db["sign_gpg"] = sign_gpg
-        if not self.db.get("security_release"):
-            self.db["security_release"] = security_release
-
         if not self.db.get("release"):
             self.db["release"] = release_tag
 
@@ -263,7 +259,6 @@ def __init__(
         print(f"- SSH username: {self.db['ssh_user']}")
         print(f"- SSH key: {self.db['ssh_key'] or 'Default'}")
         print(f"- Sign with GPG: {self.db['sign_gpg']}")
-        print(f"- Security release: {self.db['security_release']}")
         print()
 
     def checkpoint(self) -> None:
@@ -1004,34 +999,18 @@ def wait_until_all_files_are_in_folder(db: ReleaseShelf) -> None:
         are_windows_files_there = f"python-{release}.exe" in all_files
         are_macos_files_there = f"python-{release}-macos11.pkg" in all_files
         are_linux_files_there = f"Python-{release}.tgz" in all_files
-
-        if db["security_release"]:
-            # For security releases, only check Linux files
-            are_all_files_there = are_linux_files_there
-        else:
-            # For regular releases, check all platforms
-            are_all_files_there = (
-                are_linux_files_there
-                and are_windows_files_there
-                and are_macos_files_there
-            )
-
+        are_all_files_there = (
+            are_linux_files_there and are_windows_files_there and are_macos_files_there
+        )
         if not are_all_files_there:
             linux_tick = "✅" if are_linux_files_there else "❌"
             windows_tick = "✅" if are_windows_files_there else "❌"
             macos_tick = "✅" if are_macos_files_there else "❌"
-            if db["security_release"]:
-                print(
-                    f"\rWaiting for files: Linux {linux_tick} (security release mode - only checking Linux) ",
-                    flush=True,
-                    end="",
-                )
-            else:
-                print(
-                    f"\rWaiting for files: Linux {linux_tick}  Windows {windows_tick}  Mac {macos_tick} ",
-                    flush=True,
-                    end="",
-                )
+            print(
+                f"\rWaiting for files: Linux {linux_tick}  Windows {windows_tick}  Mac {macos_tick} ",
+                flush=True,
+                end="",
+            )
             time.sleep(1)
     print()
 
@@ -1396,13 +1375,6 @@ def _api_key(api_key: str) -> str:
         help="Path to the SSH key file to use for authentication",
         type=str,
     )
-    parser.add_argument(
-        "--security-release",
-        dest="security_release",
-        action="store_true",
-        default=False,
-        help="Indicate this is a security release (only checks for Linux files)",
-    )
     args = parser.parse_args()
 
     auth_key = args.auth_key or os.getenv("AUTH_INFO")
@@ -1498,7 +1470,6 @@ def _api_key(api_key: str) -> str:
         ssh_user=args.ssh_user,
         sign_gpg=not no_gpg,
         ssh_key=args.ssh_key,
-        security_release=args.security_release,
         tasks=tasks,
     )
     automata.run()