Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[ament_cmake_uncrustify] Add ament_cmake_uncrustify_LANGUAGE variable #384

Open
wants to merge 1 commit into
base: rolling
Choose a base branch
from
Open

[ament_cmake_uncrustify] Add ament_cmake_uncrustify_LANGUAGE variable #384

wants to merge 1 commit into from

Commits on May 24, 2022

  1. [ament_cmake_uncrustify] Add ament_cmake_uncrustify_LANGUAGE variable 

    This standardizes the interface to manually specify the language for
linters such as cppcheck and uncrustify, such that the necessary CMake
code can look like:

```diff
diff --git a/CMakeLists.txt b/CMakeLists.txt
index f6a8bb3..f71e803 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -19,7 +19,7 @@ if(BUILD_TESTING)

   set(ament_cmake_cppcheck_LANGUAGE c++)
-  set(ament_cmake_uncrustify_ADDITIONAL_ARGS "--language c++")
+  set(ament_cmake_uncrustify_LANGUAGE c++)

 endif()
```

Signed-off-by: Abrar Rahman Protyasha <[email protected]>
    @aprotyas
    aprotyas committed May 24, 2022
    Configuration menu
    Copy the full SHA
    682fe64 View commit details
    Browse the repository at this point in the history