cle: change run command --wait option

This changes the wait option to use the more typical --wait/--no-wait
pattern instead of having to supply a boolean string value.

Author: dlech

CHANGELOG.md

@@ -10,10 +10,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Support for Python 3.9.
 
 ### Changed
-- Update to Bleak v0.12.0
+- Update to Bleak v0.12.0.
+- Change `pybricksdev run` to use `--wait`/`--no-wait` instead of `--wait=False`.
 
 ### Fixed
-- Fix `pybricks lwp3 repl` can only connect to remote control.
+- Fix `pybricksdev lwp3 repl` can only connect to remote control.
 - Fix Technic Large hub Bluetooth hub kind.
 
 ## [1.0.0-alpha.9] - 2021-05-27

pybricksdev/cli/__init__.py

@@ -109,13 +109,28 @@ def add_parser(self, subparsers: argparse._SubParsersAction):
             "Bluetooth device name or Bluetooth address for BLE connection; "
             "serial port name for USB connection",
         )
-        parser.add_argument(
-            "--wait",
-            help="Await program completion (True) or disconnect immediately (False)",
-            required=False,
-            default="True",
-            choices=["True", "False"],
-        )
+
+        if hasattr(argparse, "BooleanOptionalAction"):
+            # BooleanOptionalAction requires Python 3.9
+            parser.add_argument(
+                "--wait",
+                help="wait for the program to complete before disconnecting",
+                action=argparse.BooleanOptionalAction,
+                default=True,
+            )
+        else:
+            parser.add_argument(
+                "--wait",
+                help="wait for the program to complete before disconnecting (default)",
+                action="store_true",
+                default=True,
+            )
+            parser.add_argument(
+                "--no-wait",
+                help="disconnect as soon as program is done downloading",
+                action="store_false",
+                dest="wait",
+            )
 
     async def run(self, args: argparse.Namespace):
         from ..ble import find_device
@@ -165,7 +180,7 @@ async def run(self, args: argparse.Namespace):
         # Connect to the address and run the script
         await hub.connect(device_or_address)
         try:
-            await hub.run(script_path, args.wait == "True")
+            await hub.run(script_path, args.wait)
         finally:
             await hub.disconnect()