workgroup:codequality:start
Differences
This shows you the differences between two versions of the page.
| Both sides previous revision Previous revision Next revision | Previous revision Last revision Both sides next revision | ||
|
workgroup:codequality:start [2017/04/20 03:58] Daniel Agar |
workgroup:codequality:start [2017/04/20 04:04] Daniel Agar |
||
|---|---|---|---|
| Line 3: | Line 3: | ||
| The Code Quality WG is focused on ways to provide incremental improvements to Dronecode project code quality and testing. | The Code Quality WG is focused on ways to provide incremental improvements to Dronecode project code quality and testing. | ||
| + | |||
| + | __Tools__ | ||
| * Coverity - https://scan.coverity.com/projects/px4-firmware | * Coverity - https://scan.coverity.com/projects/px4-firmware | ||
| * clang-tidy | * clang-tidy | ||
| - | * astyle | + | * astyle - consistent code formatting improves readability and reduces potential merge conflicts for downstream adopters. |
| __Build System__ | __Build System__ | ||
| * travis-ci (http://travis-ci.org/PX4/Firmware) | * travis-ci (http://travis-ci.org/PX4/Firmware) | ||
| + | * responsible for building and deploying production binaries for QGroundControl | ||
| * semaphoreci (https://semaphoreci.com/px4/firmware/) | * semaphoreci (https://semaphoreci.com/px4/firmware/) | ||
| * circleci (https://circleci.com/gh/PX4/Firmware) | * circleci (https://circleci.com/gh/PX4/Firmware) | ||
| * jenkins (http://sitl01.dronetest.io/) | * jenkins (http://sitl01.dronetest.io/) | ||
| + | * runs a set of multicopter and VTOL mission tests | ||
| __Testing__ | __Testing__ | ||
| Line 29: | Line 33: | ||
| * hardware tests | * hardware tests | ||
| * perf tracking | * perf tracking | ||
| + | * Automated testing (static analysis, etc) | ||
| + | * Running with address sanitier and thread sanitizer enabled | ||
| + | * Clang tidy expand usage (modernize c++) | ||
| + | * Identifying code in desperate need of refactoring (commander) | ||
| + | * Using safe C++ types | ||
| + | * A time in ms should be a time_ms type vs an int, safe addition, multiplication can then be used | ||
| + | |||
workgroup/codequality/start.txt ยท Last modified: 2017/07/27 20:43 by Mark Charlebois
Page Tools
