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
remove warning comments of stable implementation #3399
Conversation
f0b9bf8
to
8c0a383
Compare
Codecov Report
@@ Coverage Diff @@
## master #3399 +/- ##
=======================================
Coverage 85.36% 85.36%
=======================================
Files 135 135
Lines 9928 9928
=======================================
Hits 8475 8475
Misses 1068 1068
Partials 385 385 Continue to review full report at Codecov.
|
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.
I'm ok with removing some of them, but the ConnectionState
has undergone repeated change.
interface.go
Outdated
Context() context.Context | ||
// ConnectionState returns basic details about the QUIC connection. | ||
// It blocks until the handshake completes. | ||
// Warning: This API should not be considered stable and might change soon. |
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.
Don't remove this one please.
8c0a383
to
90d182a
Compare
These were implemented 4 or 5 years ago, so they actually seem to be stable.
git blame: https://github.com/lucas-clemente/quic-go/blame/master/interface.go