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

Add reflection runtime hints for OptionalValidatorFactoryBean. #28937

Closed
wants to merge 1 commit into from
Closed

Add reflection runtime hints for OptionalValidatorFactoryBean. #28937

wants to merge 1 commit into from

Conversation

OlgaMaciaszek
Copy link
Contributor

No description provided.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Aug 8, 2022
@sdeleuze sdeleuze self-assigned this Aug 8, 2022
@sdeleuze sdeleuze added theme: aot An issue related to Ahead-of-time processing and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Aug 8, 2022
@sdeleuze sdeleuze added this to the 6.0.0-M6 milestone Aug 8, 2022
@sdeleuze sdeleuze removed their assignment Aug 8, 2022
@sdeleuze sdeleuze added the status: waiting-for-triage An issue we've not yet triaged or decided on label Aug 8, 2022
@sdeleuze sdeleuze removed this from the 6.0.0-M6 milestone Aug 8, 2022
@sdeleuze
Copy link
Contributor

sdeleuze commented Aug 8, 2022

@snicoll is currently trying to check if we can remove the reflection usage here and for similar patterns, so I restored the waiting-for-triage label.

@snicoll
Copy link
Member

snicoll commented Aug 8, 2022

Thank you very much for reporting the problem and the PR @OlgaMaciaszek. I've removed the need for reflection so the additional hint shouldn't be necessary. See #28939

@snicoll snicoll closed this Aug 8, 2022
@snicoll snicoll added status: superseded An issue that has been superseded by another and removed status: waiting-for-triage An issue we've not yet triaged or decided on theme: aot An issue related to Ahead-of-time processing labels Aug 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: superseded An issue that has been superseded by another
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants