New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enable clang-tidy and cppcheck #9

Closed

3 tasks done

floatplane opened this issue Jan 16, 2024 · 0 comments

Owner

floatplane commented Jan 16, 2024 •

edited

Loading

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

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

floatplane mentioned this issue

Fix/suppress clang-tidy warnings #17

Merged

floatplane closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment