-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SendPrimitiveAsync throws InvalidOperationException for enums #291
SendPrimitiveAsync throws InvalidOperationException for enums #291
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Before merging
- update changelog in repository root.
- update version at
kiota-dotnet/Directory.Build.props
Line 4 in 0bc980f
<VersionPrefix>1.9.8</VersionPrefix>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for porting the PR here! We'll need to merge #292 before we can merge this one in. But besides that it looks good to me.
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Vincent Biret <[email protected]>
1eabf75
to
982faa6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding the trimming testing project!
tests/trimming/Microsoft.Kiota.Trimming.Validation/Microsoft.Kiota.Trimming.Validation.csproj
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for the continuous work on this one. @andrueastman for final review and merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Look good to me.
Really appreciate the work here @MartinM85
Closes #274