Skip to content
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

Make own properties for declared only fields and properties configurable #1948

Merged
merged 1 commit into from
Aug 18, 2024

Conversation

lahma
Copy link
Collaborator

@lahma lahma commented Aug 17, 2024

No description provided.

@lahma lahma changed the title Try ClrType.GetProperties(BindingFlags | BindingFlags.DeclaredOnly) Make own properties for declared only method and properties configurable Aug 18, 2024
@lahma
Copy link
Collaborator Author

lahma commented Aug 18, 2024

As the earlier change was breaking change as users could rely on having also public fields being part of JSON serialization, we need to revert that and make this declared-only behavior configurable.

/cc @lofcz

@lahma lahma enabled auto-merge (squash) August 18, 2024 06:31
@lahma lahma merged commit 780e01f into sebastienros:main Aug 18, 2024
3 checks passed
@lahma lahma changed the title Make own properties for declared only method and properties configurable Make own properties for declared only fields and properties configurable Aug 18, 2024
@lahma lahma deleted the testing-problem branch August 18, 2024 08:53
@lofcz
Copy link
Contributor

lofcz commented Aug 18, 2024

Thank you!

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