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

feat: add NotNullIfNotNull attribute on generated methods, support it on user implemented methods #1718

Merged
merged 1 commit into from
Feb 17, 2025

Conversation

latonz
Copy link
Contributor

@latonz latonz commented Feb 12, 2025

Fixes #1709
Related #1614

@latonz latonz added the enhancement New feature or request label Feb 12, 2025
@latonz latonz self-assigned this Feb 12, 2025
@latonz latonz force-pushed the 1614-nullable-attributes branch 2 times, most recently from e3fc534 to 33e9bfe Compare February 12, 2025 11:44
@latonz latonz changed the title feat: add NotNullIfNotNull attribute on generated methods feat: add NotNullIfNotNull attribute on generated methods, support it on user implemented methods Feb 12, 2025
@latonz latonz force-pushed the 1614-nullable-attributes branch 4 times, most recently from 43ee375 to 7bb382c Compare February 13, 2025 07:56
@latonz latonz closed this Feb 13, 2025
@latonz latonz reopened this Feb 13, 2025
@latonz latonz force-pushed the 1614-nullable-attributes branch 16 times, most recently from 974bb17 to 738fa07 Compare February 17, 2025 06:52
@latonz latonz marked this pull request as ready for review February 17, 2025 06:56
… on user implemented methods

If nullable attributes are not available,
the attribute is not emitted and
@latonz latonz force-pushed the 1614-nullable-attributes branch from 738fa07 to 3b217fc Compare February 17, 2025 06:57
@latonz latonz merged commit 78be0f5 into riok:main Feb 17, 2025
18 checks passed
@latonz latonz deleted the 1614-nullable-attributes branch February 17, 2025 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Duplicate mappings created when using MapDerivedType and nullable is disable
1 participant