Skip to content

fix: add support for Positive, PositiveOrZero, Negative, and NegativeOrZero validation constraints#5162

Merged
daniel-kmiecik merged 3 commits intomasterfrom
fix/positive-positiveorzero-jakarta-validation
Apr 27, 2026
Merged

fix: add support for Positive, PositiveOrZero, Negative, and NegativeOrZero validation constraints#5162
daniel-kmiecik merged 3 commits intomasterfrom
fix/positive-positiveorzero-jakarta-validation

Conversation

@daniel-kmiecik
Copy link
Copy Markdown
Collaborator

@daniel-kmiecik daniel-kmiecik commented Apr 24, 2026

Pull Request

Description

Fixes: #5035

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • ♻️ Refactor (non-breaking change)
  • 🧪 Tests
  • 📝 Documentation
  • 🧹 Chore (build or tooling)

Checklist

  • I have added/updated tests as needed
  • I have added/updated documentation where applicable
  • The PR title is descriptive
  • The code builds and passes tests locally
  • I have linked related issues (if any)

Screenshots / Additional Context

@Mattias-Sehlstedt
Copy link
Copy Markdown
Contributor

@Neo1228 also had some extra considerations for different combinations of joint annotations in #5052 that might be worth including too.

@ewaostrowska
Copy link
Copy Markdown
Contributor

Please add add tests for:

  1. added constraints with @min, @max, @DecimalMin, and @DecimalMax
  2. inclusive/exclusive flags on the OrZero annotations

@daniel-kmiecik daniel-kmiecik merged commit db9958e into master Apr 27, 2026
7 checks passed
@daniel-kmiecik daniel-kmiecik deleted the fix/positive-positiveorzero-jakarta-validation branch April 27, 2026 12:33
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.

[Bug]: Jakarta @Positive validation is not present in Swagger

3 participants