Add linter and ensure that code is warning-free

lf-lang / reactor-c

A reactor runtime written in C

Other

10 stars 24 forks source link

Add linter and ensure that code is warning-free #268

Open lhstrh opened 1 year ago

lhstrh commented 1 year ago

(...) [we] could just copy the clang-format and clang-tidy setup from reactor-cpp.

Originally posted by @cmnrd in https://github.com/lf-lang/lingua-franca/issues/1806#issuecomment-1573842961

petervdonovan commented 8 months ago

Bumping this as a reaction to #323, which is an example of a PR that mixes changes to complex code with (lots of) formatting improvements. Manual formatting improvements are great, but we can reduce the risk of missing important changes amidst formatting-related noise if we automate formatting before embarking on more big PRs.