NevynUK commented on issue #16278:
URL: https://github.com/apache/nuttx/issues/16278#issuecomment-2957689576

   > 10. **Static Analysis into CI**
   > 
   > Using static analysis we can find simple mistakes that might have been 
introduced. Things like a function that forgot to return a value in some code 
path, or use of uninitialized variable, can be caught by static analysis.
   
   Do you have an analyser in mind?
   
   I started using `cpp-check` a few years ago and have it integrated into 
several ESP32 development pipelines using a GitHub action.  Simple `Dockerfile` 
and a bash script coupled with an action.
   
   I imagine a project the size of NuttX would complicate the script but it 
should not be be too difficult (famous last words).
   
   I also added a `clang-format` checks to the same pipeline script.
   
   The two together mean that any PR should have basic analysis and style 
checks performed before it gets to review.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to