sebawagner opened a new pull request #69: URL: https://github.com/apache/openmeetings/pull/69
Enables checkstyle with 1 checkstyle check: * Unused Imports (https://checkstyle.sourceforge.io/config_imports.html#UnusedImports) It will fail at compile time for this one. I removed a few unused imports that checkstyle found. I also enabled the report. But since the only check we have will error at compile time the report will be empty, cause it will fail before generating the report. However we may add more checks later that are just warnings. Then the report becomes more useful. Please have a look what other modules/checks you would like to enable as error or warnings: https://checkstyle.sourceforge.io/checks.html For instance: * https://checkstyle.sourceforge.io/config_whitespace.html#FileTabCharacter can be quite annoying * https://checkstyle.sourceforge.io/config_misc.html#Indentation format code This can save a lot of time when reviewing a PR. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
