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

Add Preview Support for GHE endpoints #239

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rickardl
Copy link
Contributor

@rickardl rickardl commented Oct 23, 2020

This PR has a draft proposal for enabling support for Drafts (shadow-cat-preview) and Checks for GitHub Enterprise endpoints, it's an attempt to resolve the issues outlined in #235

It implements this through a change in the http.Transport implementation by extending it with required headers.

  • application/vnd.github.antiope-preview+json
  • application/vnd.github.shadow-cat-preview+json
  • etc ...

It doesn't currently offer any easy method to add further additions for V3 or V4, but there should be an optional field perform, if desired, in that case through new options.

  • V4PreviewSchemas (list)
  • V3PreviewSchemas (list)

I would appreciate feedback on this behavior, before I make further development investments.

@rickardl rickardl requested a review from itsdalmo October 23, 2020 00:14
@jlamb1
Copy link

jlamb1 commented Oct 27, 2020

Thanks for working on this @rickardl and @itsdalmo!

If it's breaking for anyone, you can specify tag: v0.21.0 in the resource to get it back up and running

@krisiye
Copy link

krisiye commented Nov 4, 2021

We have been using this against GHE for a few months now without issues. Possible to get this merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants