Skip to content

Conversation

@kurnakovv
Copy link
Contributor

@kurnakovv kurnakovv commented Sep 20, 2025

#673 issue
Second try :)

Thanks so much for your help @jsquire and @christothes in #675
I fixed my mistake and ran this Invoke-CodeGen script.

P.S. I guess I shouldn't have included the changes in the tspCodeModel.json file, right? :))
image

@kurnakovv
Copy link
Contributor Author

@jsquire @christothes @joseharriaga @trrwilson @ShivangiReja Hello everyone! This parameter is quite urgent for us, as it's crucial for our upcoming release. If anyone could fit in a quick review, it would be incredibly helpful.

Thank you so much in advance for your time and assistance! ❤️

@christothes
Copy link
Collaborator

P.S. I guess I shouldn't have included the changes in the tspCodeModel.json file, right? :))

The tspCodeModel.json file changes any time generated code changes, so you should be able to merge with main and then just regenerate. In addition, please run /scripts/export-Api.ps1 to update the public API listing.

@kurnakovv kurnakovv force-pushed the feature-673-second-try branch from 3861db2 to 61540f5 Compare September 26, 2025 05:47
@kurnakovv
Copy link
Contributor Author

kurnakovv commented Sep 26, 2025

@christothes I run this script and got this error

&: D:\C#\openai-dotnet\scripts\Export-Api.ps1:218 Line | 218 | & genapi @genapiArgs | ~~~~~~ | The term 'genapi' is not recognized as a name of a cmdlet, function, script file, or executable program. Check | the spelling of the name, or if a path was included, verify that the path is correct and try again. Cleaning up D:\C#\openai-dotnet\api\OpenAI.netstandard2.0.cs...

I install this library https://github.com/MigleSur/GenAPI

And now got this

parse-options: unknown option -- assembly parse-options: unknown option -- output-path parse-options: unknown option -- assembly-reference parse-options: unknown option -- assembly-reference parse-options: unknown option -- assembly-reference parse-options: unknown option -- assembly-reference parse-options: unknown option -- assembly-reference ERROR: blast is not installed or not included in the path.

Can you help me, please? :)
What Am I doing wrong?

P.S. I'm a little confused because blast is something from Linux, although the scripts are for powershell (Windows)

P.P.S. Okay, I realized that this repository is something completely different from what he asks for, but in any case, I didn’t find on the Internet how to replace it + I also didn’t see any script with this name in the repository

@kurnakovv kurnakovv force-pushed the feature-673-second-try branch from 6c8a3fd to c8e8f2d Compare September 26, 2025 13:30
@kurnakovv
Copy link
Contributor Author

Ok, I finally solve the problem :)
Here right tool to run script https://github.com/dotnet/sdk/blob/main/src/Compatibility/GenAPI/README.md

@jsquire
Copy link
Collaborator

jsquire commented Sep 26, 2025

Thanks very much for your contribution, @kurnakovv. Would you please be so kind as to add a change log entry to capture this?

@kurnakovv
Copy link
Contributor Author

kurnakovv commented Sep 27, 2025

@christothes Thank you so much for your approval 👍

@jsquire Yes, no problem! I’ve updated the CHANGELOG file.
One small question: should I replace [Unreleased] with 2.6.0 (2025-09-27), or is versioning handled on your side?

@jsquire
Copy link
Collaborator

jsquire commented Sep 28, 2025

@christothes Thank you so much for your approval 👍

@jsquire Yes, no problem! I’ve updated the CHANGELOG file. One small question: should I replace [Unreleased] with 2.6.0 (2025-09-27), or is versioning handled on your side?

Thanks, @kurnakovv! I've massaged this into the unreleased format that we use. I'll get this merged in.

@jsquire jsquire merged commit 1fa5728 into openai:main Sep 28, 2025
1 check passed
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