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
[confighttp] Update public methods to accept context.Context #9807
Comments
Hello @TylerHelmuth can this issue be assigned to me? And also just to confirm what I am working on, the public functions in this package are:
|
@AkhigbeEromo I agree that those are 3 three functions to add context to. |
Okay @TylerHelmuth , i will start working on this issue. Can it be assigned to me? |
Added context.Context to the following functions: ToClient ToServer ToListener Link: #9807
Closed by #9895 |
This isn't complete until we go through the deprecation cycle. We need to:
|
Is there anything i will need to do? @TylerHelmuth |
@TylerHelmuth fair enough, apologies for closing too eagerly |
@AkhigbeEromo yup, over the next 2 release cycles you can work on those tasks I listed out. For now we wait for the next release. |
Understood, Thank you @TylerHelmuth |
No description provided.
The text was updated successfully, but these errors were encountered: