From 75e18e9405c61e9108c26cda81e58b9eb6c06048 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?F=C3=A9lix=20Pi=C3=A9dallu?= <felix@piedallu.me>
Date: Thu, 15 Feb 2024 22:52:11 +0100
Subject: [PATCH] autoupdate_app_sources: set proper defaults on argparse, just
 to be sure

---
 tools/autoupdate_app_sources/autoupdate_app_sources.py | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/tools/autoupdate_app_sources/autoupdate_app_sources.py b/tools/autoupdate_app_sources/autoupdate_app_sources.py
index d8c675c..03c962d 100755
--- a/tools/autoupdate_app_sources/autoupdate_app_sources.py
+++ b/tools/autoupdate_app_sources/autoupdate_app_sources.py
@@ -504,9 +504,12 @@ def main() -> None:
     parser = argparse.ArgumentParser()
     parser.add_argument("apps", nargs="*", type=Path,
                         help="If not passed, the script will run on the catalog. Github keys required.")
-    parser.add_argument("--edit", action=argparse.BooleanOptionalAction, help="Edit the local files", default=True)
-    parser.add_argument("--commit", action=argparse.BooleanOptionalAction, help="Create a commit with the changes")
-    parser.add_argument("--pr", action=argparse.BooleanOptionalAction, help="Create a pull request with the changes")
+    parser.add_argument("--edit", action=argparse.BooleanOptionalAction, default=True,
+                        help="Edit the local files")
+    parser.add_argument("--commit", action=argparse.BooleanOptionalAction, default=False,
+                        help="Create a commit with the changes")
+    parser.add_argument("--pr", action=argparse.BooleanOptionalAction, default=False,
+                        help="Create a pull request with the changes")
     parser.add_argument("--paste", action="store_true")
     parser.add_argument("-j", "--processes", type=int, default=multiprocessing.cpu_count())
     args = parser.parse_args()