-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
HBASE-28689 Add more files in spotless for trim tailing whitespaces #6020
Conversation
There are two commits, first the commit is to change the spotless config, and then second one is for apply the spotless rule. |
💔 -1 overall
This message was automatically generated. |
I can reproduce the error locally with jdk11, but with jdk17 everything is fine... Let me check whether the latest jdk11 can solve the problem... |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
Newest jdk11 still has the NPE, and strange that it also failed when building against master branch... Digging... |
OK, the problem is that, we need to run a Let me check how to add this in our yetus scripts... |
(!) A patch to the testing environment has been detected. |
3 similar comments
(!) A patch to the testing environment has been detected. |
(!) A patch to the testing environment has been detected. |
(!) A patch to the testing environment has been detected. |
💔 -1 overall
This message was automatically generated. |
Seems does not work... Let me send an email to yetus dev list to ask how to do this... And since blanks check are now optional, will close this PRs first. Once I found a way to deal with mvn site problem, will open PRs again. Thanks. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
No description provided.