You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It would be convenient if there was a way to exclude one or more files from lint checks when invoking ament_auto_lint.
This would be useful in circumstances were a package includes third-party source files (e.g. tinydir in ros2/rcl#332).
Currently, there is no apparent way to exclude a file other than not running a particular lint check.
Linters could be called separately (ie. not with ament_auto_lint), but some would still need to be extended to support file exclusions.
Implementation considerations
Ideally, we could pass a file exclusion list to ament_auto_lint and let it pass the appropriate flags to the linters it invokes, e.g.
Another option could be to support placing a file (e.g. ament_lint.cfg) in the root of the repo that allows configuration of the various linters supported by ament_lint.
The text was updated successfully, but these errors were encountered:
# this must happen before the invocation of ament_package() if(BUILD_TESTING)
find_package(ament_lint_auto REQUIRED)
set(AMENT_LINT_AUTO_FILE_EXCLUDE /path/to/ignored_file ...)
ament_lint_auto_find_test_dependencies()
endif()
It would be convenient if there was a way to exclude one or more files from lint checks when invoking
ament_auto_lint
.This would be useful in circumstances were a package includes third-party source files (e.g. tinydir in ros2/rcl#332).
Currently, there is no apparent way to exclude a file other than not running a particular lint check.
Linters could be called separately (ie. not with
ament_auto_lint
), but some would still need to be extended to support file exclusions.Implementation considerations
Ideally, we could pass a file exclusion list to
ament_auto_lint
and let it pass the appropriate flags to the linters it invokes, e.g.Another option could be to support placing a file (e.g.
ament_lint.cfg
) in the root of the repo that allows configuration of the various linters supported by ament_lint.The text was updated successfully, but these errors were encountered: