Skip to content

Fix simplified name for ValueTuple<T>#2033

Merged
AArnott merged 2 commits intoMessagePack-CSharp:developfrom
AlanLiu90:Branch_FixSimplifiedName
Oct 23, 2024
Merged

Fix simplified name for ValueTuple<T>#2033
AArnott merged 2 commits intoMessagePack-CSharp:developfrom
AlanLiu90:Branch_FixSimplifiedName

Conversation

@AlanLiu90
Copy link
Contributor

Fix two issues:

  1. ValueTuple<int> results invalid generated code: (int)
  2. Some primitive types are not handled in QualifiedNamedTypeName.GetQualifiedName: float, double, decimal, bool, object

@AArnott AArnott added this to the v3.0 milestone Oct 23, 2024
Copy link
Collaborator

@AArnott AArnott left a comment

Choose a reason for hiding this comment

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

Thank you for this. Looks like I should spin another RC right away to fix this regression.

@AArnott AArnott enabled auto-merge October 23, 2024 03:39
@AArnott AArnott merged commit 168179e into MessagePack-CSharp:develop Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants