Refine nullability of `Assert#noNullElements` by ngocnhan-tran1996 · Pull Request #35868 · spring-projects/spring-framework

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@ngocnhan-tran1996

Copy link

Contributor

@ngocnhan-tran1996 ngocnhan-tran1996 commented

Nov 22, 2025

edited

Loading

From reference

@Nullable Object @Nullable [] array means both individual elements and the array can be null.

https://docs.spring.io/spring-framework/reference/core/null-safety.html

I think we should to add @Nullable because both the array and its elements may be null.

Signed-off-by: Tran Ngoc Nhan <ngocnhan.tran1996@gmail.com>

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage

An issue we've not yet triaged or decided on

label

Nov 22, 2025

@sdeleuze sdeleuze self-assigned this

Nov 24, 2025

@sdeleuze sdeleuze added in: core

Issues in core modules (aop, beans, core, context, expression)

type: enhancement

A general enhancement

and removed status: waiting-for-triage

An issue we've not yet triaged or decided on

labels

Nov 24, 2025

@sdeleuze sdeleuze added this to the 7.0.2 milestone

Nov 24, 2025

@sdeleuze sdeleuze changed the title Add @Nullable to clarify nullability Refine nullability of Assert#noNullElements

Nov 24, 2025

@sdeleuze sdeleuze closed this in c599775

Nov 24, 2025

@sdeleuze

Copy link

Contributor

Merged, thanks!

@ngocnhan-tran1996 ngocnhan-tran1996 deleted the add-nullable branch

November 24, 2025 09:48

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

Reviewers

No reviews

Assignees

@sdeleuze sdeleuze

Labels

in: core

Issues in core modules (aop, beans, core, context, expression)

type: enhancement

A general enhancement

Projects

None yet

Milestone

7.0.2

Development

Successfully merging this pull request may close these issues.

3 participants

@ngocnhan-tran1996 @sdeleuze @spring-projects-issues