Skip to content

Remove null support from getWrapperType input, add test coverage #4571

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

Merged
merged 1 commit into from
May 24, 2025

Conversation

scordio
Copy link
Contributor

@scordio scordio commented May 24, 2025

Overview

Initially triggered by #4557 (comment).


I hereby agree to the terms of the JUnit Contributor License Agreement.


Definition of Done

@scordio scordio force-pushed the reflection-utils branch 2 times, most recently from 5971581 to a54d332 Compare May 24, 2025 15:56
@scordio scordio changed the title Annotate getWrapperType parameter as @Nullable, add test coverage Remove null support from getWrapperType input, add test coverage May 24, 2025
@scordio scordio force-pushed the reflection-utils branch from a54d332 to 404a628 Compare May 24, 2025 16:09
@scordio scordio marked this pull request as ready for review May 24, 2025 16:13
@scordio scordio force-pushed the reflection-utils branch from 404a628 to 76d8c68 Compare May 24, 2025 16:14
@marcphilipp marcphilipp merged commit 1f5d97e into junit-team:main May 24, 2025
13 checks passed
@marcphilipp
Copy link
Member

Thanks, @scordio! 👍

@scordio scordio deleted the reflection-utils branch May 24, 2025 18:12
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.

2 participants