Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Change callbacks to virtual from being abstract#39526

Merged
davidfowl merged 3 commits intomasterfrom
davidfowl/pipe-events
Jul 16, 2019
Merged

Change callbacks to virtual from being abstract#39526
davidfowl merged 3 commits intomasterfrom
davidfowl/pipe-events

Conversation

@davidfowl
Copy link
Copy Markdown
Member

  • Obsolete the callbacks as well

Fixes #38362

- Obsolete the callbacks as well
Co-Authored-By: Stephen Toub <stoub@microsoft.com>
@davidfowl
Copy link
Copy Markdown
Member Author

fyi @mgravell @AArnott

@mgravell
Copy link
Copy Markdown

"This will be removed in a future release." - I'm willing to bet it never will be, but I think that messaging is necessary to suitably dissuade folks

@davidfowl davidfowl merged commit 51cd784 into master Jul 16, 2019
@karelz karelz added this to the 3.0 milestone Jul 16, 2019
@stephentoub stephentoub deleted the davidfowl/pipe-events branch July 23, 2019 14:55
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deprecate PipeWriter.OnReaderCompleted

5 participants