fixing issue with backing field naming and serialization failing by epitka · Pull Request #1785 · MessagePack-CSharp/MessagePack-CSharp · GitHub
Skip to content

fixing issue with backing field naming and serialization failing#1785

Merged
AArnott merged 2 commits into
MessagePack-CSharp:masterfrom
epitka:master
Mar 31, 2024
Merged

fixing issue with backing field naming and serialization failing#1785
AArnott merged 2 commits into
MessagePack-CSharp:masterfrom
epitka:master

Conversation

@epitka

@epitka epitka commented Mar 8, 2024

Copy link
Copy Markdown

Fix for #1776

@AArnott AArnott requested a review from neuecc March 14, 2024 12:44

@AArnott AArnott 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.

@neuecc I'm not familiar with the changed code. Would you mind taking a look?

@epitka

epitka commented Mar 14, 2024

Copy link
Copy Markdown
Author

@epitka

epitka commented Mar 21, 2024

Copy link
Copy Markdown
Author

Any chance of somebody taking a look at it?

@AArnott AArnott 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.

Thanks for the added explanation. I'll take it.

@AArnott AArnott enabled auto-merge March 31, 2024 22:19
@AArnott AArnott merged commit 43f1b1c into MessagePack-CSharp:master Mar 31, 2024
@epitka-sg

Copy link
Copy Markdown
Contributor

@AArnott How do I get a hold of package that has this fix merged in? Latest released is 2.5.140?

@AArnott AArnott added this to the v2.5 milestone Jun 6, 2024
@AArnott

AArnott commented Jun 6, 2024

Copy link
Copy Markdown
Collaborator

This was referenced Jun 15, 2026
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.

3 participants