search

lf-lang / lingua-franca

Intuitive concurrent programming in any language
https://www.lf-lang.org
Other
236 stars 62 forks source link

Use spotbugs for static code analysis and make the code base warning free #1806

Open cmnrd opened 1 year ago

cmnrd commented 1 year ago

Now that we have formatting addressed, we should also introduce linting and enforce that the code is warning free. Gradle has a convenient plugin called checkstyle for linting java code. It supports various style guide rule sets, including Google's style guide. This should be a tremendous help in avoiding the most common errors such as the problems fixed in https://github.com/lf-lang/lingua-franca/pull/1793. The biggest challenge, however, will be to fix all linter warnings in our existing code.

lhstrh commented 1 year ago

Would be cool to set this up for reactor-c as well...

cmnrd commented 1 year ago

For reactor-c you could just copy the clang-format and clang-tidy setup from reactor-cpp.