-
Notifications
You must be signed in to change notification settings - Fork 0
Prerequisite org.openrewrite.java.migrate.lang.var.UseVarFor... #11446
#1
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
base: main
Are you sure you want to change the base?
Conversation
| recipeList: | ||
| - org.openrewrite.staticanalysis.EqualsAvoidsNull | ||
| # - org.openrewrite.java.migrate.lang.var.UseVarForGenericMethodInvocations # bug; | ||
| # - org.openrewrite.java.migrate.lang.var.UseVarForGenericsConstructors # 700 changes... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kindly asking if this is something to consider? @gavinking
- a couple of tiny optimizations, and use of 'var' hibernate/hibernate-orm#11446
- https://docs.openrewrite.org/recipes/java/migrate/lang/var/usevarforgenericsconstructors
I am wondering whether applying this change at scale would end up impacting performance levels in any meaningful or beneficial way, or how it actually plays out in practice.
It seems like it would align the codebase very well when applied consistently.
The use of var is also being seen in other projects, like junit:
|
No, we're not interested. And this is exactly the sort of pull request in which it's easy to hide a supply chain attack. |
hibernate-testing/src/main/java/org/hibernate/testing/orm/junit/DomainModelExtension.java
Fixed
Show fixed
Hide fixed
Thanks for feedback. What you mean with attack? To sneak something in? By me or the plugin? Its just some dome and the changes are up to you to recreate and merge in your own if needed. This plugin and recipe and active in checkstyle and the rule covered by checkstyle as well is active in junit so idk exactly where the issue is. Thanks for considering. |
|
spotless is outdated in the current state, despite the major update there are not changes. Might still be worth update and check dependabot. |
…ernate#11446 Signed-off-by: Vincent Potucek <[email protected]>
|
kindly asking where the attack is now? the codebase is well formatted, having achieved conventional principle. Thanks. @gavinking |
Prerequisite
org.openrewrite.java.migrate.lang.var.UseVarFor...hibernate#11446[Please describe here what your change is about]
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.