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

RequiredByDefault is not applied correctly to dto object #179

Closed
wants to merge 5 commits into from

Conversation

cl3m
Copy link
Contributor

@cl3m cl3m commented Feb 28, 2024

Where required by default is true, the data classes properties are not required by default which make all the properties nullable.

@Carapacik
Copy link
Owner

@cl3m Make sure that your code passes all checks, in this case format it.

@Carapacik Carapacik self-requested a review February 28, 2024 20:42
@Carapacik Carapacik requested review from Carapacik and removed request for Carapacik February 29, 2024 18:19
@Carapacik Carapacik added bug Something isn't working duplicate This issue or pull request already exists invalid This doesn't seem right labels Mar 3, 2024
@Carapacik Carapacik closed this Mar 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants