Skip to content
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

docs: clarify "deprecated" to indicate whether some features may be removed in 1.x #2900

Merged
merged 3 commits into from
Jul 11, 2019

Conversation

dfawley
Copy link
Member

@dfawley dfawley commented Jul 10, 2019

Also, deprecate WithBalancerName (was experimental).

The decision of whether something may be removed is based on if it was marked experimental as of v1.0/GA or upon its introduction, in accordance with https://github.com/grpc/grpc-go/blob/master/Documentation/versioning.md.

… removed

or not based on whether they were marked experimental as of v1.0/GA.
@dfawley dfawley added the Type: Documentation Documentation or examples label Jul 10, 2019
@dfawley dfawley requested a review from menghanl July 10, 2019 19:40
dialoptions.go Outdated
// https://github.com/grpc/grpc/blob/master/doc/service_config.md
//
// Deprecated: service config should be received through name resolver, as
// specified here. Will be removed in a future 1.x release.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where?

@dfawley dfawley merged commit 1290649 into grpc:master Jul 11, 2019
@dfawley dfawley added this to the 1.23 Release milestone Jul 11, 2019
@dfawley dfawley deleted the deprecate_this branch July 12, 2019 16:45
@lock lock bot locked as resolved and limited conversation to collaborators Jan 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Documentation Documentation or examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants