cli: Use required=False instead of bogus default values for @click.ar…

…gument This fixes a bug with click8, where picomc play <version> would not work because it expected a boolean argument.
sammko · Jun 3, 2021 · d8bbf43 · d8bbf43
1 parent 64285b3
commit d8bbf43
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/picomc/cli/account.py b/picomc/cli/account.py
@@ -29,7 +29,7 @@ def _list(am):
 
 @account_cli.command()
 @account_cmd
-@click.argument("mojang_username", default="")
+@click.argument("mojang_username", required=False)
 @pass_account_manager
 def create(am, account, mojang_username):
     """Create an account."""

diff --git a/picomc/cli/play.py b/picomc/cli/play.py
@@ -7,7 +7,7 @@
 
 
 @click.command()
-@click.argument("version", default=False)
+@click.argument("version", required=False)
 @click.option("-a", "--account", "account_name")
 @click.option("--verify", is_flag=True, default=False)
 @pass_instance_manager