lingua-franca icon indicating copy to clipboard operation
lingua-franca copied to clipboard

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

Open cmnrd opened this issue 1 year ago • 2 comments

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.

cmnrd avatar Jun 01 '23 07:06 cmnrd

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

lhstrh avatar Jun 02 '23 05:06 lhstrh

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

cmnrd avatar Jun 02 '23 14:06 cmnrd