Enable clang-tidy and cppcheck - Githubissues

floatplane / MitsuQTT

Use ESP8266/ESP32 hardware to control Mitsubishi heat pumps via MQTT

GNU Lesser General Public License v2.1

3 stars 2 forks source link

Enable clang-tidy and cppcheck #9

Closed floatplane closed 9 months ago

floatplane commented 9 months ago

This should be possible but I haven't gotten it working yet. #8 shows a bug that should have gotten caught by clang-tidy.

Breadcrumbs

https://clang.llvm.org/extra/clang-tidy/checks/bugprone/assignment-in-if-condition.html https://docs.platformio.org/en/latest/advanced/static-code-analysis/index.html https://docs.trunk.io/

TODO

[x] Enable the checkers (fixed in #10)
[x] Fix the warnings
[x] Add to github actions and fail if future warnings occur