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

Use Go stdlib context package instead of golang.org/x/net/context #11187

Merged
merged 1 commit into from Apr 29, 2020

Conversation

tklauser
Copy link
Member

The context package was added in Go 1.7 [1]. The implementation in
x/net/context is merely a type alias now on newer Go versions, so use
context directly.

[1] https://golang.org/doc/go1.7#context

The context package was added in Go 1.7 [1]. The implementation in
x/net/context is merely a type alias now on newer Go versions, so use
context directly.

[1] https://golang.org/doc/go1.7#context

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
@tklauser tklauser added pending-review kind/cleanup This includes no functional changes. release-note/misc This PR makes changes that have no direct user impact. labels Apr 28, 2020
@tklauser tklauser requested review from a team as code owners April 28, 2020 11:16
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.8.0 Apr 28, 2020
@tklauser
Copy link
Member Author

test-me-please

@coveralls
Copy link

Coverage Status

Coverage increased (+0.007%) to 44.725% when pulling 5e0ae19 on pr/tklauser/replace-x-net-context into 904cae6 on master.

@tklauser tklauser added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. and removed pending-review labels Apr 28, 2020
@qmonnet qmonnet merged commit dc49144 into master Apr 29, 2020
1.8.0 automation moved this from In progress to Merged Apr 29, 2020
@qmonnet qmonnet deleted the pr/tklauser/replace-x-net-context branch April 29, 2020 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cleanup This includes no functional changes. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.8.0
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

5 participants