Skip to content

Fix issue #79 duplicate validation errors#84

Merged
DamianEdwards merged 3 commits intomainfrom
dedwards/0.10-fix-79-duplicate-errors
May 4, 2026
Merged

Fix issue #79 duplicate validation errors#84
DamianEdwards merged 3 commits intomainfrom
dedwards/0.10-fix-79-duplicate-errors

Conversation

@DamianEdwards
Copy link
Copy Markdown
Owner

Fixes #79

Summary

  • Ignore validation attributes surfaced through TypeDescriptor when they are equivalent to attributes already read from reflection.
  • Preserve distinct validation attributes, including TypeDescriptor-attached attributes.
  • Refresh cached type details when TypeDescriptor providers change.

Testing

  • dotnet test tests\MiniValidation.UnitTests\MiniValidation.UnitTests.csproj -f net8.0 --no-restore --verbosity minimal
  • dotnet build src\MiniValidation\MiniValidation.csproj --no-restore --verbosity minimal

DamianEdwards and others added 3 commits May 3, 2026 20:56
Avoid adding validation attributes surfaced by TypeDescriptor when they duplicate attributes already read from reflection, while preserving distinct reflected and TypeDescriptor-attached validators.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@DamianEdwards DamianEdwards mentioned this pull request May 4, 2026
@DamianEdwards DamianEdwards merged commit 64be1f5 into main May 4, 2026
1 check passed
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.

Validation error is returned twice

1 participant