Skip to content

[ament_cmake_lint_cmake] Add file exclude support#1

Merged
guillaumeautran merged 3 commits into
jazzyfrom
ga_jazzy_cmake_lint
Jun 25, 2025
Merged

[ament_cmake_lint_cmake] Add file exclude support#1
guillaumeautran merged 3 commits into
jazzyfrom
ga_jazzy_cmake_lint

Conversation

@guillaumeautran

Copy link
Copy Markdown
Collaborator

This is a local backport of the upstream PR#546. It's only intention is to live until the upstream PR is merged.

In the `ament_lint_cmake` CMake function, the optional list
argument `EXCLUDE` can now be used as an exclusion specifier.
The `ament_cmake_lint_cmake` respects the
`AMENT_LINT_AUTO_FILE_EXCLUDE` variable.

Signed-off-by: Guillaume Autran <gautran@clearpath.ai>
The `ament_cmake_pep257` respects the `AMENT_LINT_AUTO_FILE_EXCLUDE` variable.

Signed-off-by: Guillaume Autran <gautran@clearpath.ai>
The `ament_cmake_xmllit` respects the `AMENT_LINT_AUTO_FILE_EXCLUDE` variable.

Signed-off-by: Guillaume Autran <gautran@clearpath.ai>
@guillaumeautran guillaumeautran self-assigned this Jun 25, 2025
@guillaumeautran guillaumeautran requested review from jhiggins-cpr and tonybaltovski and removed request for tonybaltovski June 25, 2025 13:19
@guillaumeautran guillaumeautran merged commit 0141ebf into jazzy Jun 25, 2025
@guillaumeautran guillaumeautran deleted the ga_jazzy_cmake_lint branch June 25, 2025 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants