Add NewTokenClient

[icholy]

Fixes #2636

[codecov]

Codecov Report

Merging #2637 (3548305) into master (2561c07) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #2637   +/-   ##
=======================================
  Coverage   98.05%   98.05%           
=======================================
  Files         130      130           
  Lines       11242    11244    +2     
=======================================
+ Hits        11023    11025    +2     
  Misses        150      150           
  Partials       69       69           

Impacted Files	Coverage Δ
github/github.go	97.46% <100.00%> (+<0.01%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[icholy]

I removed the context parameter since I think most users won't be needing it. Let me know if you disagree.

[gmlewis]

I removed the context parameter since I think most users won't be needing it. Let me know if you disagree.

Yeah, one thing I learned at Google is that people sometimes do wild and wacky things with the context.
Let's please put it back in.

[gmlewis]

Thank you, @icholy !
LGTM.

Awaiting second LGTM+Approval from any other contributor to the repo.

[valbeat]

LGTM!

[icholy]

Should I squash?

[gmlewis]

Should I squash?

No need to squash, as we always "Squash and Merge" all PRs in this repo. Thanks, though.

[gmlewis]

Do you want to update an example before merging, @icholy ?

[gmlewis]

LGTM!

Thank you, @valbeat !

[icholy]

I'll do a separate PR later.

[gmlewis]

I'll do a separate PR later.

Sounds great. Merging.