Adds support for serialization of csv values for Go enums#3210
Conversation
5d44c0e to
e9914cc
Compare
|
@rkodev multiple integration tests are failing, can you have a look please? |
549fa21 to
219bfd4
Compare
95c25d3 to
dd29479
Compare
dd29479 to
3ffc4ab
Compare
|
Kudos, SonarCloud Quality Gate passed! |
andrueastman
left a comment
There was a problem hiding this comment.
@baywet I'm okay to have this merged as it fits into the general bigger picture of what we'd want to do, and I've documented the conversations/findings at #3237.
@rkodev You can probably assign the related Go SDK issues to me and then I'll close them once the issue is closed (hopefully before the next generation cycle 😅 ).
baywet
left a comment
There was a problem hiding this comment.
I'm perfectly fine with merging this one as is and have the work on the flags being implemented later. I know this won't solve the customer issue that's linked right now, but we're making progress towards it. @rkodev maybe unlink the "fixes" before you merge the PR?








Resolves microsoftgraph/msgraph-sdk-go#517
Adds support for serialization of csv values for Go enums by adding a value mask that can support multiple Go values