Skip to content

Conversation

@erik-madryga
Copy link

No description provided.

… and change GH_MODEL_ID fallback to a model that actually exists
@github-actions
Copy link
Contributor

👋 Thanks for contributing @erik-madryga! We will review the pull request and get back to you soon.

@github-actions github-actions bot requested a review from koreyspace November 16, 2025 13:43
@erik-madryga erik-madryga changed the title fix(framework): changing path for people who install dotnet with brew and change GH_MODEL_ID fallback to a model that actually exists fix(framework): change GH_MODEL_ID fallback to a model that actually exists Nov 16, 2025
@koreyspace
Copy link
Contributor

@justinyoo can you take a quick look at this? gpt-5-mini is a model listed on the GitHub model marketplace but this change is noting that "it doesn't exist".

@erik-madryga erik-madryga changed the title fix(framework): change GH_MODEL_ID fallback to a model that actually exists fix(framework): change GH_MODEL_ID fallback to a model that could be found Nov 26, 2025
@erik-madryga
Copy link
Author

@justinyoo can you take a quick look at this? gpt-5-mini is a model listed on the GitHub model marketplace but this change is noting that "it doesn't exist".

sorry for the sassy commit message, I thought I was committing to my repo when I wrote this, but yeah I wasn't able to get this to work with gpt-5-mini

@justinyoo
Copy link
Contributor

@justinyoo can you take a quick look at this? gpt-5-mini is a model listed on the GitHub model marketplace but this change is noting that "it doesn't exist".

sorry for the sassy commit message, I thought I was committing to my repo when I wrote this, but yeah I wasn't able to get this to work with gpt-5-mini

Hmmmm interesting. Have you tried it with GitHub Codespaces or devcontainer?

I use both Windows and MacOS, and this works on both machines. It also works on GH codespace instance using gpt-5-mini.

I'll close this PR.

@justinyoo justinyoo closed this Nov 26, 2025
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.

3 participants