search

jlab-sensing / soil-power-sensor-firmware

Firmware for STM32 and ESP32 microcontrollers on the Soil Power Sensor board
MIT License
0 stars 3 forks source link

Add linting and static code analysis #78

Closed jmadden173 closed 1 month ago

jmadden173 commented 2 months ago

Name/Affiliation/Title John

Purpose of the PR Incremental code to #59. Adds static code analysis in the form of cppcheck and cpplint. Errors have not been fixed. Add vscode tasks for easy running before push. Also removed the default_env to allow checking for building of all envs.

Development Environment Linux spruce 6.8.2-arch2-1 #1 SMP PREEMPT_DYNAMIC Thu, 28 Mar 2024 17:06:35 +0000 x86_64 GNU/Linux PlatformIO Core, version 6.1.7 Used ST-Link V3 MINIE

Test Procedure See the builds for this PR.

Additional Context When merging don't close issue or delete branch.

stevegtaylor commented 1 month ago

Can you walk me through how this is supposed to look/work when we're actually writing code?