Skip to content

build: Support manually generating C# bindings via Github action and add a README - #423

Merged
anakrish merged 1 commit into
microsoft:mainfrom
dekomissMSFT:dekomiss/bindingstrigger
Jul 3, 2025
Merged

build: Support manually generating C# bindings via Github action and add a README#423
anakrish merged 1 commit into
microsoft:mainfrom
dekomissMSFT:dekomiss/bindingstrigger

Conversation

@dekomissMSFT

Copy link
Copy Markdown
Collaborator

Update the Github action that generates C# bindings for Regorus to support manual generation. This allows for easily generating Nuget packages for any branch, which is useful for integration testing of in-progress Regorus changes with other projects during development.

Also update the Readme for the C# bindings to explain how to generate a nuget package.

Comment thread .github/workflows/test-csharp.yml Outdated
Comment thread .github/workflows/test-csharp.yml Outdated
@anakrish

anakrish commented Jul 2, 2025

Copy link
Copy Markdown
Collaborator

@dekomissMSFT The clippy errors are unrelated to your changes. The rusttool chain has moved ahead and clippy is now flagging more code to rewrite.
I have a PR to fix those: #424

@anakrish

anakrish commented Jul 3, 2025

Copy link
Copy Markdown
Collaborator

@dekomissMSFT The PR looks good. Please squash your commits and fix the commit message.

I've been also following the conventional commit approach of tagging the commit message. Then the commits will be grouped automatically in the release description: https://github.com/microsoft/regorus/releases

@dekomissMSFT dekomissMSFT changed the title Support manually generating C# bindings via Github action and add a README feat: Support manually generating C# bindings via Github action and add a README Jul 3, 2025
@dekomissMSFT dekomissMSFT changed the title feat: Support manually generating C# bindings via Github action and add a README build: Support manually generating C# bindings via Github action and add a README Jul 3, 2025
@dekomissMSFT
dekomissMSFT force-pushed the dekomiss/bindingstrigger branch from c98aa72 to 062effe Compare July 3, 2025 16:59
@dekomissMSFT

Copy link
Copy Markdown
Collaborator Author

@dekomissMSFT The PR looks good. Please squash your commits and fix the commit message.

I've been also following the conventional commit approach of tagging the commit message. Then the commits will be grouped automatically in the release description: https://github.com/microsoft/regorus/releases

Fixed

@anakrish anakrish left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@anakrish
anakrish merged commit 168b2a9 into microsoft:main Jul 3, 2025
33 checks 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.

2 participants