Skip to content
New issue

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

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

Already on GitHub? # to your account

Improve package with catkin_lint #89

Merged
merged 1 commit into from Aug 22, 2018
Merged

Improve package with catkin_lint #89

merged 1 commit into from Aug 22, 2018

Conversation

VictorLamoine
Copy link
Contributor

Before

$ catkin_lint -W3 .
rviz_visual_tools: notice: target name 'imarker_simple_demo' might not be sufficiently unique
rviz_visual_tools: CMakeLists.txt(15): notice: list COMPONENTS should be sorted
rviz_visual_tools: CMakeLists.txt(35): notice: operands for operator VERSION_LESS should be quoted strings
rviz_visual_tools: CMakeLists.txt(54): notice: list CATKIN_DEPENDS should be sorted
rviz_visual_tools: CMakeLists.txt(86): notice: list of source files should be sorted
rviz_visual_tools: CMakeLists.txt(149): notice: list TARGETS should be sorted
rviz_visual_tools: CMakeLists.txt(173): notice: list TARGETS should be sorted
catkin_lint: checked 1 packages and found 7 problems

After

$ catkin_lint -W3 .
rviz_visual_tools: notice: target name 'imarker_simple_demo' might not be sufficiently unique
rviz_visual_tools: CMakeLists.txt(86): notice: list of source files should be sorted
rviz_visual_tools: CMakeLists.txt(173): notice: list TARGETS should be sorted
catkin_lint: checked 1 packages and found 3 problems

@davetcoleman
Copy link
Member

Thanks!

@davetcoleman davetcoleman merged commit e76afa2 into PickNikRobotics:melodic-devel Aug 22, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants