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

fix: handle baseUrl suffix for GHES #186

Merged
merged 4 commits into from
Sep 9, 2020
Merged

Conversation

gr2m
Copy link
Contributor

@gr2m gr2m commented Sep 9, 2020

@gr2m gr2m added the Type: Bug Something isn't working as documented label Sep 9, 2020
@gr2m gr2m force-pushed the 185/handle-ghes-baseurl-suffix branch from 9a5f7d8 to d35209a Compare September 9, 2020 17:33
test/defaults.test.ts Outdated Show resolved Hide resolved
src/graphql.ts Outdated Show resolved Hide resolved
@lencioni
Copy link
Contributor

lencioni commented Sep 9, 2020

Thanks for working on this!

gr2m and others added 2 commits September 9, 2020 11:21
Co-authored-by: Joe Lencioni <joe.lencioni@gmail.com>
Co-authored-by: Joe Lencioni <joe.lencioni@gmail.com>
@gr2m gr2m merged commit fd2b7bf into master Sep 9, 2020
@gr2m gr2m deleted the 185/handle-ghes-baseurl-suffix branch September 9, 2020 18:25
@github-actions
Copy link

github-actions bot commented Sep 9, 2020

🎉 This PR is included in version 4.5.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

gr2m added a commit to octokit/core.js that referenced this pull request Oct 20, 2020
….graphql()` and GHES

The customized baseUrl passed to `@octokit/graphql` is there for legacy reasons. It is no longer necessary since octokit/graphql.js#186
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Something isn't working as documented
Projects
None yet
2 participants