[Improvement]: Add checkstyle check for wildcard import
Search before asking
- [X] I have searched in the issues and found no similar issues.
What would you like to be improved?
throw exception if there is any wildcard import when building
How should we improve?
We can add checkstyle plugin for this.
Why not add new spotless rule for this?
- currently spotless can only use google java format 1.7(because this is the highest version support JDK8 )
- google java format did not ban wildcard import in 1.7
- spotless did not support custom configure for maven(only supported in gradle)
Are you willing to submit PR?
- [X] Yes I am willing to submit a PR!
Subtasks
No response
Code of Conduct
- [X] I agree to follow this project's Code of Conduct
Thank you very much for suggesting this improvement. It is indeed very important and can help us simplify the code review process, allowing us to focus more on logic rather than code formatting.
Adding an additional checkstyle plugin is acceptable, considering that currently spotless seems unable to achieve this. I'm curious whether the checkstyle plugin can help us validate if our code depends on shaded classes.
@zhoujinsong thanks for your reply, we can add rule for shaded class allowed only, such as flink