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

3.x: Fix wrong description for bean validation annotations (#8667) #8681

Conversation

Captain1653
Copy link
Contributor

Description

Resolves #8667

I followed Jakarta Bean Validation Specification

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Apr 18, 2024
@@ -191,6 +193,7 @@ Supported types are:
|`@Size`
a|The annotated element size must be between the specified boundaries (included).
Supported types are:

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Without this empty line content looks a little wrong.
Example:
Screenshot 2024-04-18 at 18 49 37

With this empty line:
Screenshot 2024-04-18 at 18 50 46

@Captain1653
Copy link
Contributor Author

@spericas Can you review backport for 3.x? Please :)

@Captain1653
Copy link
Contributor Author

Some checks are failed, but I didn't touch them.
Screenshot 2024-04-26 at 07 16 57

@barchetta
Copy link
Member

@Captain1653 you'll need to rebase this PR with the latest from the helidon-3.x branch.

The example build failure is caused by the combination of examples having moved out of this repository and the fact that we did a 3.x release and bumped the 3.x version while this PR has been open. So you need to pick up the pom version change to 3.2.9-SNAPHOST.

@Captain1653 Captain1653 force-pushed the 3.x-8687-beanvalidation-wrong-description branch from 9f07295 to 1b2bd9c Compare May 1, 2024 20:12
@Captain1653
Copy link
Contributor Author

@barchetta Thank you for the explanation.

Done.

@romain-grecourt romain-grecourt merged commit 90d9391 into helidon-io:helidon-3.x May 2, 2024
12 checks passed
@Captain1653 Captain1653 deleted the 3.x-8687-beanvalidation-wrong-description branch May 2, 2024 03:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants