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
fix: add SendWithContext function #92
Conversation
6e86d8e
to
580572d
Compare
Out of curiosity, why is this supported back to go1.6? Aren't only the last 2 versions officially supported by the go team? |
@@ -6,6 +6,8 @@ import ( | |||
"io/ioutil" | |||
"net/http" | |||
"net/url" | |||
|
|||
"golang.org/x/net/context" |
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.
Since it is 2021 now, at this point, let's use "context" and drop support for Go <= 1.7?
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.
Thank you @someone1! 🎆
Fixes
[fixes #91]
Checklist
Short description of what this PR does: