dubeejw commented on a change in pull request #2848: Support reading param.json
from stdin #2837
URL:
https://github.com/apache/incubator-openwhisk/pull/2848#discussion_r145748713
##########
File path: tools/cli/go-whisk-cli/commands/action.go
##########
@@ -1060,11 +1060,11 @@ func init() {
actionUpdateCmd.Flags().StringSliceVarP(&flags.common.annotation,
"annotation", "a", []string{}, wski18n.T("annotation values in `KEY VALUE`
format"))
actionUpdateCmd.Flags().StringVarP(&flags.common.annotFile,
"annotation-file", "A", "", wski18n.T("`FILE` containing annotation values in
JSON format"))
actionUpdateCmd.Flags().StringSliceVarP(&flags.common.param, "param", "p",
[]string{}, wski18n.T("parameter values in `KEY VALUE` format"))
- actionUpdateCmd.Flags().StringVarP(&flags.common.paramFile, "param-file",
"P", "", wski18n.T("`FILE` containing parameter values in JSON format"))
+ actionUpdateCmd.Flags().StringVarP(&flags.common.paramFile, "param-file",
"P", "", wski18n.T("`FILE` containing parameter values in JSON format, use '-'
to read from stdin"))
actionUpdateCmd.Flags().StringVar(&flags.action.web, WEB_FLAG, "",
wski18n.T("treat ACTION as a web action, a raw HTTP web action, or as a
standard action; yes | true = web action, raw = raw HTTP web action, no | false
= standard action"))
actionInvokeCmd.Flags().StringSliceVarP(&flags.common.param, "param", "p",
[]string{}, wski18n.T("parameter values in `KEY VALUE` format"))
- actionInvokeCmd.Flags().StringVarP(&flags.common.paramFile, "param-file",
"P", "", wski18n.T("`FILE` containing parameter values in JSON format"))
+ actionInvokeCmd.Flags().StringVarP(&flags.common.paramFile, "param-file",
"P", "", wski18n.T("`FILE` containing parameter values in JSON format, use '-'
to read from stdin"))
Review comment:
You will need to update the user-facing message in the exported message file.
https://github.com/apache/incubator-openwhisk/blob/master/tools/cli/go-whisk-cli/wski18n/resources/en_US.all.json#L711
https://github.com/apache/incubator-openwhisk/blob/master/tools/cli/go-whisk-cli/wski18n/resources/en_US.all.json#L712
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
