Clean up checkstyle warnings#2765
Conversation
LinusDietz
left a comment
There was a problem hiding this comment.
Hi @snisnisniksonah, thank you for your contribution.
Improving the readability of the code is indeed something useful!
While I honor the contribution, I would kindly ask you to change the destination of this PR to JabRef's master branch instead of checkstyle. (unless, of course, you have been in contact with @koppor before)
Furthermore, please execute gradle check to check that your changes don't break our build which seems to be the case:
:checkstyleJmh FAILED
FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':checkstyleJmh'.
> Unable to create a Checker: configLocation {/tmp/checkstyle/config/checkstyle/checkstyle.xml},
| private static final Pattern WHITESPACE_AROUND_NEWLINE = Pattern.compile("(?m)^\\s+|\\v+"); | ||
| private static final Pattern DOUBLE_WHITESPACES = Pattern.compile("[ ]{2,}"); | ||
| private final String htmlText; | ||
|
|
There was a problem hiding this comment.
I don't see why there should be a blank line
Fix all checkstyle warnings and added checkstyle suppression in gen
2358cfe to
d3dfd46
Compare
…into checkstyle # Conflicts: # src/main/java/org/jabref/gui/entryeditor/FieldExtraComponents.java # src/main/java/org/jabref/preferences/JabRefPreferences.java
|
@snisnisniksonah, I have added a [WIP] tag to the title to indicate that this PR is still work in progress. Please remove it when you want feedback for this PR. |
LinusDietz
left a comment
There was a problem hiding this comment.
I think these changes are fine. Just update the version number of checkstyle and it can be merged.
| // do not use other packages for checkstyle, excluding gen(erated) sources | ||
| checkstyleMain.source = "src/main/java" | ||
| toolVersion = '6.17' | ||
| toolVersion = '7.6' |
There was a problem hiding this comment.
The latest version is already 7.6.1.
|
@snisnisniksonah Could you try to adapt the IntelliJ code style accordingly (https://github.com/JabRef/jabref/tree/master/config)? |
Clean up checkstyle warnings
Add checkstyle suppression for gen