Skip to content
This repository was archived by the owner on Jul 10, 2025. It is now read-only.

Conversation

@jacobperron
Copy link
Collaborator

Style changes to conform to the new default setting introduced in ament/ament_lint#210.
Arguments that do not fit on one line must start on a new line.

Style changes to conform to the new default setting introduced in ament/ament_lint#210.
Arguments that do not fit on one line must start on a new line.

Signed-off-by: Jacob Perron <[email protected]>
@jacobperron
Copy link
Collaborator Author

Linux: Build Status

@chapulina
Copy link
Contributor

It looks like there was an infrastructure issue before. New build: Build Status

Copy link
Contributor

@chapulina chapulina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! The uncrustify errors are gone:

17:13:51 5: -- run_test.py: verify result file '/home/jenkins-agent/workspace/ci_linux/ws/build/gazebo_plugins/test_results/gazebo_plugins/uncrustify.xunit.xml'
17:13:51  5/23 Test  #5: uncrustify ..............................   Passed    1.34 sec
17:13:02 7: -- run_test.py: verify result file '/home/jenkins-agent/workspace/ci_linux/ws/build/gazebo_ros/test_results/gazebo_ros/uncrustify.xunit.xml'
17:13:02  7/19 Test  #7: uncrustify .......................   Passed    0.82 sec

@chapulina chapulina merged commit 8198324 into ros2 Mar 14, 2020
@chapulina chapulina deleted the jacob/uncrustify branch March 14, 2020 03:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants