-
Notifications
You must be signed in to change notification settings - Fork 40.5k
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
Polish #42192
Polish #42192
Conversation
arefbehboudi
commented
Sep 9, 2024
- Removed @SuppressWarnings as it was no longer needed.
- Used pattern matching.
- Updated documentation.
- Replaced class with record.
...a/org/springframework/boot/rsocket/context/RSocketPortInfoApplicationContextInitializer.java
Outdated
Show resolved
Hide resolved
...ng-boot-project/spring-boot/src/main/java/org/springframework/boot/json/JsonValueWriter.java
Outdated
Show resolved
Hide resolved
...-boot-project/spring-boot/src/main/java/org/springframework/boot/jdbc/DataSourceBuilder.java
Outdated
Show resolved
Hide resolved
@wilkinsona Thanks for the review. I've updated the code based on your feedback. Please check it again when you can. |
@arefbehboudi thanks for the contribution. It wasn't building as the formatting was no longer correct. FTR you could have seen that locally by building the module, something like:
I've polished things in ece5c6f |
@snicoll Thanks for pointing that out! I'll make sure to run the build locally using the provided command to catch any formatting issues in the future. Thanks again for the guidance! |