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

Remove redundant conditions and enhance safety with List.of() method #32673

Closed
wants to merge 3 commits into from

Conversation

dukbong
Copy link
Contributor

@dukbong dukbong commented Apr 19, 2024

  • Revised the portion where null check was performed after Assert.notNull(), eliminating redundant null checks
  • Enhanced code safety by replacing Arrays.asList() with List.of() to create immutable lists

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Apr 19, 2024
@dukbong dukbong closed this Apr 19, 2024
@bclozel bclozel added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Apr 19, 2024
@sbrannen
Copy link
Member

@dukbong, please refrain from opening PRs and then closing them and replacing them with additional PRs, since this introduces unnecessary noise in the issue tracker.

@dukbong dukbong deleted the reduce branch April 19, 2024 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: declined A suggestion or change that we don't feel we should currently apply
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants