DaanHoogland merged PR #91:
URL: https://github.com/apache/cloudstack-go/pull/91
--
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-unsubscr...@cloudst
Pearl1594 commented on PR #91:
URL: https://github.com/apache/cloudstack-go/pull/91#issuecomment-2546826849
Thanks @phsm - tests now run successfully! Thank you :+1:
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
phsm commented on PR #91:
URL: https://github.com/apache/cloudstack-go/pull/91#issuecomment-2545978224
@Pearl1594 This was to tidy up an abandoned https://github.com/golang/mock/
package.
The authors there suggest to switch to a maintained fork instead.
I replaced the rest of the
phsm commented on code in PR #91:
URL: https://github.com/apache/cloudstack-go/pull/91#discussion_r1887053442
##
generate/layout.go:
##
@@ -790,16 +864,92 @@ var layout = apiInfo{
"RegistrationService": {
"registerOauthProvider",
},
- "Shutd
Pearl1594 commented on PR #91:
URL: https://github.com/apache/cloudstack-go/pull/91#issuecomment-2545895377
For some reason, the tests seem to fail on this PR - @phsm can you please
help us with this:
I needed to make the following changes to make it work - I am not sure why
the 1st two
Pearl1594 commented on code in PR #91:
URL: https://github.com/apache/cloudstack-go/pull/91#discussion_r1886985264
##
generate/layout.go:
##
@@ -790,16 +864,92 @@ var layout = apiInfo{
"RegistrationService": {
"registerOauthProvider",
},
- "
phsm commented on PR #91:
URL: https://github.com/apache/cloudstack-go/pull/91#issuecomment-2545574592
@DaanHoogland updated both the code, and the PR header/description.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use