-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Regenerate client from 7.0.1 #15
Conversation
@@ -19,7 +19,7 @@ Name | Type | Description | Notes | |||
|
|||
### NewHostactivationJoinToken | |||
|
|||
`func NewHostactivationJoinToken() *HostactivationJoinToken` | |||
`func NewHostactivationJoinToken(name string, ) *HostactivationJoinToken` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is unexpected for this PR. Also name is supposed to be required. But previously it was generated with name as optional. Having this change will require the terraform provider to be fixed as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed, also update Terraform Provider.
@param ctx context.Context - for authentication, logging, cancellation, deadlines, tracing, etc. Passed from http.Request or context.Background(). | ||
@return ApiKeysKerberosPostRequest | ||
*/ | ||
KeysKerberosPost(ctx context.Context) ApiKeysKerberosPostRequest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is unexpected for this PR.
Maybe Post method was excluded from the schema, the first time it was generated, and the changes were not pushed.
go fmt
go fmt
again