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

Fix test errors in 3x3 covariance test cases #2000

Merged
merged 1 commit into from
Oct 10, 2024
Merged

Fix test errors in 3x3 covariance test cases #2000

merged 1 commit into from
Oct 10, 2024

Conversation

leocencetti
Copy link
Contributor

Fix errors in tests introduced by #1998

@vooon

vooon
vooon approved these changes Oct 10, 2024
View reviewed changes
Copy link
Member

@vooon vooon left a comment

Choose a reason for hiding this comment

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

Thanks!

@vooon vooon added this to the Version 2.9 milestone Oct 10, 2024
@vooon vooon merged commit 166e832 into mavlink:ros2 Oct 10, 2024
0 of 4 checks passed
@leocencetti
Copy link
Contributor Author

@vooon This is embarassing. Apparently uncrustify now complains. Unfortunately I currently can't build the tests locally, so couldn't check. I'll try to get something running...

@leocencetti leocencetti deleted the fix-test-errors branch October 10, 2024 07:51
@vooon
Copy link
Member

vooon commented Oct 10, 2024

@leocencetti don't mind. It always complains. Different releases has different rules, which makes me mad...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vooon vooon vooon approved these changes

Assignees
No one assigned
Labels
fix ros2
Projects
ROS2 support
Status: Done
Milestone
Version 2.9
Development

Successfully merging this pull request may close these issues.
2 participants
@leocencetti @vooon