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

Bug [Bean Validation]: Compilation error with array of strings with validations #926

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

joschi
Copy link

@joschi joschi commented Dec 23, 2024

Fixes #925

@joschi joschi requested a review from a team as a code owner December 23, 2024 15:43
@ricardozanini
Copy link
Member

Hi! Thank you for reporting this bug. Do you plan to append this PR with the fix?

@joschi
Copy link
Author

joschi commented Dec 23, 2024

Do you plan to append this PR with the fix?

That depends a bit on my availability in the next few weeks, but definitely not this year anymore. 😅

If you're already in the code base and don't have to learn about it, I'd appreciate somebody else fixing this bug.

Didn't plan to, but added a fix.

@joschi joschi force-pushed the bean-validation-missing-import branch from 1c165d7 to 74efde2 Compare January 31, 2025 12:55
@joschi
Copy link
Author

joschi commented Jan 31, 2025

@ricardozanini I've added the missing imports to the respective template. Could you please re-run the workflows?

@hbelmiro
Copy link
Contributor

hbelmiro commented Feb 1, 2025

@joschi I'm not following. Is it supposed to be reviewed or it's just a bug report? If the latter, can you revert it to draft and add a DO NOT MERGE to the title?

@joschi
Copy link
Author

joschi commented Feb 1, 2025

@hbelmiro It's a bug reproducer with a fix added one month later. 😉

@ricardozanini
Copy link
Member

@joschi @hbelmiro actually we have another PR to fix this and avoid star imports: #978

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug [Bean Validation]: Compilation error with array of strings with validations
4 participants