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

Clang tidy add extra arg option and return 1 when any error is catched. #452

Open
wants to merge 4 commits into
base: rolling
Choose a base branch
from
Open

Clang tidy add extra arg option and return 1 when any error is catched. #452

wants to merge 4 commits into from

Commits on Feb 7, 2024

  1. feat: allow to pass extra-arg argument to clang-tidy 

    Signed-off-by: WU MENGHUNG <[email protected]>
    @wuisky
    wuisky committed Feb 7, 2024
    Configuration menu
    Copy the full SHA
    46b976e View commit details
    Browse the repository at this point in the history

  2. fix: catch error infos when clang-tidy fails, also return 1 when fails. 

    Signed-off-by: WU MENGHUNG <[email protected]>
    @wuisky
    wuisky committed Feb 7, 2024
    Configuration menu
    Copy the full SHA
    eafe421 View commit details
    Browse the repository at this point in the history

  3. feat: allow to pass multiple extra-arg arguments to clang-tidy 

    Signed-off-by: WU MENGHUNG <[email protected]>
    @wuisky
    wuisky committed Feb 7, 2024
    Configuration menu
    Copy the full SHA
    6210f2f View commit details
    Browse the repository at this point in the history

Commits on Sep 6, 2024

  1. Merge branch 'rolling' of github.com:ament/ament_lint into clang-tidy… 

    …-add-extra-arg-option
    @wuisky
    wuisky committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    1e9b71f View commit details
    Browse the repository at this point in the history