-
Notifications
You must be signed in to change notification settings - Fork 634
CASSGO-55 - User API doc enhancement #1863
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
base: trunk
Are you sure you want to change the base?
Conversation
4c82386
to
2ec07af
Compare
2ec07af
to
171d258
Compare
171d258
to
044b4f5
Compare
filters.go
Outdated
@@ -61,12 +62,6 @@ func DataCenterHostFilter(dataCenter string) HostFilter { | |||
}) | |||
} | |||
|
|||
// Deprecated: Use DataCenterHostFilter instead. | |||
// DataCentreHostFilter is an alias that doesn't use the preferred spelling. | |||
func DataCentreHostFilter(dataCenter string) HostFilter { |
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 was deprecated very recently for the 2.0 release, we don't want to remove it now.
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.
Got it, I have returned it.
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.
Added 1 comment. #1868 will change the query and batch API so I'd like to get that merged before merging this one.
044b4f5
to
63b6d78
Compare
Sorry for that, I've accidentally pushed without changes. |
Some of the user-exposed API is not documented which can cause misuse. This PR provides doc update fo the public methods and interfaces. Also, some deprecated features were removed.
Fix for the #1601