Hey fellow RIOTers, our CI currently fails on added files having either doxygen or cppcheck warnings.
Unfortunately if changed files contain either warning, it cannot determine whether the warnings are new or have been there. So I hereby launch the Code Quality Task Force, and declare it's first task to get rid of all those warnings. Let's go as follows: 1. move all vendor headers into a clearly recognizable sub directory containing "/vendor/", for easy exclusion (Hauke will do that) 2. Activate CI failing on all doxygen warnings. We'll do that in #6688. (Oleg will do that). 3. Pull request (and merge) fixes to #6688 until all warnings are gone and the PR passes CI, then merge the PR. 4. create a PR similar to #6688, but for cppcheck warnings. Same fixing procedure. 6. meet up on next RIOT Summit and have a beer together. Everyone is invited and very welcome to help at any point! :) @Oleg, do you have numbers on the amount of cppcheck/doxygen warnings we currently have? Kaspar _______________________________________________ devel mailing list devel@riot-os.org https://lists.riot-os.org/mailman/listinfo/devel
