ament_clang_tidy - Fix Reporting when WarningsAsErrors is specified i…

…n config (#397) (#488) * feature Signed-off-by: Matt Condino <[email protected]> (cherry picked from commit f985e67) Co-authored-by: Matt Condino <[email protected]>
ament · May 10, 2024 · c5e554a · c5e554a
1 parent b23ccc9
commit c5e554a
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/ament_clang_tidy/ament_clang_tidy/main.py b/ament_clang_tidy/ament_clang_tidy/main.py
@@ -171,6 +171,9 @@ def start_subprocess(full_cmd):
                 print('The invocation of "%s" failed with error code %d: %s' %
                       (os.path.basename(clang_tidy_bin), e.returncode, e),
                       file=sys.stderr)
+                # Attempt to recover output, if any was found (eg - if
+                # WarningsAsErrors was specified in the config file).
+                output = e.output.decode('utf-8')
             return output
 
         files = []