You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@cloudstack.apache.org by "shwstppr (via GitHub)" <gi...@apache.org> on 2023/03/15 12:06:04 UTC

[GitHub] [cloudstack-go] shwstppr opened a new pull request, #50: generate: fix error logging for goimports

shwstppr opened a new pull request, #50:
URL: https://github.com/apache/cloudstack-go/pull/50

   This PR adds error returned by goimports in the log.
   Currently, only output is printed,
   ```
   ⇒  go run -v generate/generate.go generate/layout.go --api=generate/listApis.json 
   command-line-arguments
   2023/03/15 17:20:50 1 API(s) failed to generate:
   2023/03/15 17:20:50 GoImport failed to format:
   exit status 1
   ```
   
   With fix,
   ```
   ⇒  go run -v generate/generate.go generate/layout.go --api=generate/listApis.json 
   command-line-arguments
   2023/03/15 17:21:24 1 API(s) failed to generate:
   2023/03/15 17:21:24 GoImport failed to format:
   
   exec: "goimports": executable file not found in $PATH
   exit status 1
   ```


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org

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

[GitHub] [cloudstack-go] rohityadavcloud merged pull request #50: generate: fix error logging for goimports

Posted by "rohityadavcloud (via GitHub)" <gi...@apache.org>.

rohityadavcloud merged PR #50:
URL: https://github.com/apache/cloudstack-go/pull/50


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@cloudstack.apache.org

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