Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

sgoedecke
Copy link
Contributor

This PR adds support for theGitHub 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.

@hyperlint-aiHyperlint AI
Copy link
Contributor

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 theHyperlint 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 thehyperlint-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 addhyperlint-ignore to the PR to ignore the link check for this PR.

Copy link
Contributor

@DouweMDouweM 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

@DouweMDouweM 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
ContributorAuthor

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

Copy link
Contributor

@DouweMDouweM left a comment

Choose a reason for hiding this comment

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

@sgoedecke New tests look good, thanks. I just noticed one final omission. Fix that and I'll merge! 🎉

sgoedecke reacted with heart emoji
@DouweMDouweM merged commitcbbd966 intopydantic:mainJul 4, 2025
19 checks passed
@DouweM
Copy link
Contributor

@sgoedecke Thanks Sean! We'll release this early next week

@sgoedecke
Copy link
ContributorAuthor

No worries. Thank you for your comments!

@DouweMDouweM mentioned this pull requestJul 4, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@DouweMDouweMDouweM requested changes

Assignees

@DouweMDouweM

Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@sgoedecke@DouweM
[8]ページ先頭
©2009-2025 Movatter.jp