Skip to content

Export extensible-enum for csharp#9879

Merged
timotheeguerin merged 2 commits intomicrosoft:mainfrom
RodgeFu:export-extensible-enum
Mar 3, 2026
Merged

Export extensible-enum for csharp#9879
timotheeguerin merged 2 commits intomicrosoft:mainfrom
RodgeFu:export-extensible-enum

Conversation

@RodgeFu
Copy link
Copy Markdown
Contributor

@RodgeFu RodgeFu commented Mar 3, 2026

This pull request addresses a missing export for the extensible enum feature in C# and refactors related type and function names to avoid confliction. The main focus is to ensure that the extensible enum component is properly exported and its interface is accurately named.

Extensible Enum Export and Refactoring:

  • Added a missing export for the extensible enum component in the C# emitter framework's index file (packages/emitter-framework/src/csharp/components/index.ts).
  • Renamed the interface from EnumDeclarationProps to ExtensibleEnumDeclarationProps in extensible-enum.tsx to avoid confiliction.
  • Updated the ExtensibleEnumDeclaration function to use the new ExtensibleEnumDeclarationProps interface.

Documentation and Change Tracking:

  • Added a change log entry documenting the fix for the missing export in the .chronus/changes/export-extensible-enum-2026-2-3-14-47-18.md file.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Mar 3, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/emitter-framework@9879

commit: e06d14f

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 3, 2026

All changed packages have been documented.

  • @typespec/emitter-framework
Show changes

@typespec/emitter-framework - fix ✏️

Add the missing export in index for extensible-enum in csharp

@azure-sdk
Copy link
Copy Markdown
Collaborator

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

@timotheeguerin timotheeguerin added this pull request to the merge queue Mar 3, 2026
Merged via the queue into microsoft:main with commit 684313e Mar 3, 2026
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emitter-framework Issues for the emitter framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants