moveit_calibration icon indicating copy to clipboard operation
moveit_calibration copied to clipboard

[WIP] Add clang-tidy test

Open tylerjw opened this issue 3 years ago • 2 comments

This is for development of this approach to running clang-tidy as it is faster to iterate here than on moveit.

tylerjw avatar Mar 24 '21 03:03 tylerjw

Codecov Report

Merging #75 (0133029) into master (3ffd736) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 0133029 differs from pull request most recent head b959746. Consider uploading reports for the commit b959746 to get more accurate results Impacted file tree graph

@@           Coverage Diff           @@
##           master      #75   +/-   ##
=======================================
  Coverage   72.75%   72.75%           
=======================================
  Files          10       10           
  Lines         554      554           
=======================================
  Hits          403      403           
  Misses        151      151           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3ffd736...b959746. Read the comment docs.

codecov[bot] avatar Mar 24 '21 03:03 codecov[bot]

This is for development of this approach to running clang-tidy as it is faster to iterate here than on moveit.

Glad to be the MoveIt guinea pig

JStech avatar Mar 24 '21 15:03 JStech

Closing. We have clang-tidy support in MoveIt2.

rhaschke avatar Aug 29 '24 18:08 rhaschke