-
Notifications
You must be signed in to change notification settings - Fork 61
docs(readme): add example for using a custom Agent #614
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
Conversation
bf027ab
to
1c68882
Compare
Co-authored-by: Gregor Martynus <[email protected]>
Co-authored-by: Gregor Martynus <[email protected]>
Co-authored-by: Gregor Martynus <[email protected]>
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.
Thanks Oscar for taking this on!
If you like, a great follow up would be to re-enable tests in @octokit/core
based on your code
See
- https://github.com/octokit/core.js/pull/563/files#diff-3b92aa641b383348adffd8b4135a36e205df863a1c555a27ee2476ecf60d8795R9
- https://github.com/octokit/core.js/pull/563/files#diff-55a29a52bf060cb89f0dd68612c5cbcdbbb95b67070eb7a2f666d2e1bfb8e2efR16
I'd start by creating an issue in @octokit/core
and then take it from there
🎉 This PR is included in version 8.1.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Resolves #613
Behavior
Before the change?
No documentation on how to set an
Agent
tooctokit.request
internalfetch
.After the change?
Documentation on how to set an
Agent
tooctokit.request
internalfetch
.Other information
Additional info
Pull request checklist
Does this introduce a breaking change?
Please see our docs on breaking changes to help!
Type: Breaking change
label)If
Yes
, what's the impact:Pull request type
Type: Documentation