fix: add required option to vefaas deploy cli (#105)

yaozheng-fang · web-flow · commit 7eae9c109948 · 2025-08-26T17:03:31.000+08:00
When user not specify `--vefaas-app-name`, a `None` value is incorrect. Use must give vefaas app name.
diff --git a/veadk/cli/cli_deploy.py b/veadk/cli/cli_deploy.py
@@ -29,7 +29,9 @@
     default=None,
     help="Volcengine secret key",
 )
-@click.option("--vefaas-app-name", help="Expected Volcengine FaaS application name")
+@click.option(
+    "--vefaas-app-name", required=True, help="Expected Volcengine FaaS application name"
+)
 @click.option(
     "--veapig-instance-name", default="", help="Expected Volcengine APIG instance name"
 )

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,9 @@`
`29`	`29`	`default=None,`
`30`	`30`	`help="Volcengine secret key",`
`31`	`31`	`)`
`32`		`-@click.option("--vefaas-app-name", help="Expected Volcengine FaaS application name")`
	`32`	`+@click.option(`
	`33`	`+ "--vefaas-app-name", required=True, help="Expected Volcengine FaaS application name"`
	`34`	`+)`
`33`	`35`	`@click.option(`
`34`	`36`	`"--veapig-instance-name", default="", help="Expected Volcengine APIG instance name"`
`35`	`37`	`)`