[GitHub] [skywalking-cli] hanahmily commented on pull request #82: chore: reorganize the project layout and add an example of .skywalking.yaml

[GitBox <gi...@apache.org>]

hanahmily commented on pull request #82:
URL: https://github.com/apache/skywalking-cli/pull/82#issuecomment-748078825


   Noted. Actually, SWCK needs functions in `pkg` with an error return value, instead of exiting the process when facing some errors as below 
   
   ```
   func AllServices(cliCtx *cli.Context, duration api.Duration) ([]api.Service, error)
   ```
   
   Do you have any convenient path to support it? 
   The only downside of the current solution is SWCK has to copy the implementation of these functions, which brings some issues if the details of them are updated.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org