-
Notifications
You must be signed in to change notification settings - Fork 25.4k
bump spotless to 7.2.1 #133023
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
bump spotless to 7.2.1 #133023
Conversation
Now working anymore after the bump. Removing the custom spotless config fixes the issue, but thats not what we want.
|
@@ -59,6 +59,7 @@ public void apply(Project project) { | |||
|
|||
java.target("src/**/*.java"); | |||
java.removeUnusedImports(); | |||
java.removeWildcardImports(); |
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.
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.
please merge
As I have to recreate it, as already mentioned. The code need to be complain to use this feature.
@DaveCTurner or do you want an even bigger PR, as rewrite its not doing it at once i have to run it multiple times.
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.
Assuming most changes are not covered by Spotless, as there are some blind spots—as its the case with every tool, including Rewrite.
It seems I cannot recreate the changes you're requesting, as they aren't achievable through a simple Spotless configuration change.
Since removing unused imports isn't a new task, would it be acceptable to add a plugin specifically for this purpose? This approach would allow for future enhancements, when there is the demand or not.
As there are a log best practises this project could benefit from including:
Closing this as its mostly a duplicate off effort we invested already in #115750 but had to explicitly roll back due to an issue in the spotless plugin I had raised in diffplug/spotless#2515 |
gradle check
?