Skip to content

Conversation

@MaiLinhP
Copy link
Contributor

Made enums internal to fix build issues

// - Renamed.
// - Made internal until we support the convenience methods.
[CodeGenType("OrderEnum")]
internal readonly partial struct InternalOrderEnum
Copy link
Collaborator

Choose a reason for hiding this comment

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

The only other tip / suggestion I have is to use the access decorator to set this type to internal. Though I don't see us using that anywhere else, so feel free to ignore and see what Jose thinks!

@jorgerangel-msft jorgerangel-msft merged commit 0241d8e into openai:typespec/update-http-client-csharp-1.0.0-alpha.20260116.3 Jan 23, 2026
2 checks passed
jorgerangel-msft pushed a commit that referenced this pull request Jan 27, 2026
…lpha.20260116.3 (#900)

* Update @typespec/http-client-csharp to 1.0.0-alpha.20260116.3

- Updated @typespec/http-client-csharp from 1.0.0-alpha.20260111.1 to 1.0.0-alpha.20260116.3
- Updated Microsoft.TypeSpec.Generator.ClientModel from 1.0.0-alpha.20260111.1 to 1.0.0-alpha.20260116.3
- Regenerated OpenAI SDK code with new generator version
- Updated centrally managed package-lock.json file with new dependency versions

* Bump to version 1.0.0-alpha.20260116.3 - Add internal partial structs for new enums (#907)

* Add internal partial structs for new enums

* Remove custom code that causes duplication in generated code

---------

Co-authored-by: Linh Phan <[email protected]>

* Delete IncludedConversationItemProperty custom code that can be generated (#919)

* Add internal partial structs for new enums

* Remove custom code that causes duplication in generated code

* Delete custom implementation that can be generated

---------

Co-authored-by: Linh Phan <[email protected]>

---------

Co-authored-by: GitHub Action <[email protected]>
Co-authored-by: Linh Phan <[email protected]>
Co-authored-by: Linh Phan <[email protected]>
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.

2 participants