Skip to content

Add deprecation to Http2StreamChannelBootstrap.open0(...) as it was m…#9372

Merged
normanmaurer merged 1 commit into4.1from
http2_deprecation
Jul 16, 2019
Merged

Add deprecation to Http2StreamChannelBootstrap.open0(...) as it was m…#9372
normanmaurer merged 1 commit into4.1from
http2_deprecation

Conversation

@normanmaurer
Copy link
Copy Markdown
Member

…arked as public by mistake

Motivation:

Mark Http2StreamChannelBootstrap.open0(...) as deprecated as the user should not use it. It was marked as public by mistake.

Modifications:

Add deprecation warning.

Result:

User will be aware the method should not be used directly.

…arked as public by mistake

Motivation:

Mark Http2StreamChannelBootstrap.open0(...) as deprecated as the user should not use it. It was marked as public by mistake.

Modifications:

Add deprecation warning.

Result:

User will be aware the method should not be used directly.
Copy link
Copy Markdown
Member

@njhill njhill left a comment

Choose a reason for hiding this comment

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

LGTM though I'm not a direct consumer of this API so may not understand all the implications.

@normanmaurer normanmaurer merged commit 4f172c1 into 4.1 Jul 16, 2019
@normanmaurer normanmaurer deleted the http2_deprecation branch July 16, 2019 11:08
@normanmaurer normanmaurer added this to the 4.1.38.Final milestone Jul 16, 2019
normanmaurer added a commit that referenced this pull request Jul 16, 2019
…arked as public by mistake (#9372)

Motivation:

Mark Http2StreamChannelBootstrap.open0(...) as deprecated as the user should not use it. It was marked as public by mistake.

Modifications:

Add deprecation warning.

Result:

User will be aware the method should not be used directly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants