mdeuser commented on a change in pull request #2326: (Review) Alphabetize
URL:
https://github.com/apache/incubator-openwhisk/pull/2326#discussion_r125700672
##########
File path: tools/cli/go-whisk-cli/commands/api.go
##########
@@ -445,8 +453,11 @@ var apiListCmd = &cobra.Command{
* Takes an API object (containing one more more single
basepath/relpath/operation triplets)
* and some filtering configuration. For each API endpoint matching the
filtering criteria, display
* each endpoint's configuration - one line per configuration property (action
name, verb, api name, api gw url)
+ * Parses and initializes values for ApiFilteredRow struct
*/
-func printFilteredListApi(resultApi *whisk.RetApi, api *whisk.ApiOptions) {
+func parseFilteredList(resultApi *whisk.RetApi, api *whisk.ApiOptions,flagType
string) []whisk.ApiFilteredList {
Review comment:
The function names is not really representative of the function behavior -
which is to generate a filtered list. There's really not much parsing.
----------------------------------------------------------------
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
