Skip to content

Polishing#30250

Closed
ZouHongxue wants to merge 0 commit intospring-projects:mainfrom
ZouHongxue:main
Closed

Polishing#30250
ZouHongxue wants to merge 0 commit intospring-projects:mainfrom
ZouHongxue:main

Conversation

@ZouHongxue
Copy link
Copy Markdown

@ZouHongxue ZouHongxue commented Mar 31, 2023

Base on SonarLint and IDEA suggestions, polishing the code.

  • Reorder the modifiers to comply with the Java Language Specification.
  • java:S4276 Refactor this code to use the more specialised Functional Interface 'UnaryOperator'
  • Merge this if statement with the enclosing one.
  • Use record for immutable classes.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Mar 31, 2023
@sdeleuze sdeleuze self-assigned this Mar 31, 2023
@ZouHongxue ZouHongxue closed this Apr 3, 2023
@ZouHongxue ZouHongxue mentioned this pull request Apr 3, 2023
Closed
@snicoll snicoll removed the status: waiting-for-triage An issue we've not yet triaged or decided on label Sep 17, 2023
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

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ZouHongxue @sdeleuze @snicoll @spring-projects-issues