ament / ament_lint

Apache License 2.0
38 stars 107 forks source link

ament_lint_cmake doesn't display reasons for error when run through pre-commit #495

Open bijoua29 opened 5 months ago

bijoua29 commented 5 months ago

ament_lint_cmake doesn't display reasons for error when run through pre-commit hook. It just displays exit code of 1 like:

ament_lint_cmake.........................................................Failed

However if I run ament_lint_cmake manually, it displays the actual errors.

mjcarroll commented 5 months ago

Can you elaborate on what you are trying to do here? I don't believe that we have a recommended pre-commit hook setup.

I imagine that what you are running into is that the ament_lint_cmake utility either outputs to stdout or stderr and that the pre-commit script is expecting the output on the opposite.