Skip to content

Add GitHub Models provider #2114

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

sgoedecke
Copy link

This PR adds support for the GitHub Models inference provider (GitHub's own inference API). It's OpenAI compatible and uses a GitHub PAT (or any token with the right permission) for authentication.

I'm pretty sure GitHub meets the 10k stars requirement for adding a new model, but please let me know if you'd prefer this be handled differently. I'm very happy to make any adjustments or changes.

Copy link
Contributor

hyperlint-ai bot commented Jul 2, 2025

PR Change Summary

Added support for GitHub Models inference provider, enhancing compatibility with OpenAI and enabling authentication via GitHub personal access tokens.

  • Introduced GitHub Models provider for inference support
  • Provided usage examples for integrating GitHub Models
  • Documented required permissions for GitHub personal access tokens
  • Listed supported model families with their prefixes

Modified Files

  • docs/models/openai.md

How can I customize these reviews?

Check out the Hyperlint AI Reviewer docs for more information on how to customize the review.

If you just want to ignore it on this PR, you can add the hyperlint-ignore label to the PR. Future changes won't trigger a Hyperlint review.

Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add hyperlint-ignore to the PR to ignore the link check for this PR.

Copy link
Contributor

@DouweM DouweM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sgoedecke Thanks Sean! I left a few comments, and can you please have a look at the failing test?

Copy link
Contributor

@DouweM DouweM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sgoedecke Thanks Sean! I left a few comments, and can you please have a look at the failing test?

@sgoedecke
Copy link
Author

@DouweM Thank you for your thoughtful comments. I've pushed up some changes and fixed up the tests.

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

Successfully merging this pull request may close these issues.

2 participants