Skip to content

[JTC] Fix deprecated header#610

Merged
bmagyar merged 3 commits intoros-controls:masterfrom
christophfroehlich:fix_includes
May 22, 2023
Merged

[JTC] Fix deprecated header#610
bmagyar merged 3 commits intoros-controls:masterfrom
christophfroehlich:fix_includes

Conversation

@christophfroehlich
Copy link
Copy Markdown
Member

qos_event.hpp got deprecated:

This header is obsolete, please include rclcpp/event_handler.hpp instead

See ros2/rclcpp#2069

Copy link
Copy Markdown
Member

@destogl destogl left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Copy Markdown
Member

@bmagyar bmagyar left a comment

Choose a reason for hiding this comment

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

Sanity check: can't this go on master directly?

@christophfroehlich
Copy link
Copy Markdown
Member Author

christophfroehlich commented May 13, 2023 via email

@bmagyar bmagyar changed the base branch from jtc-features to master May 13, 2023 08:47
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 13, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 35.98%. Comparing base (e7f9962) to head (3853f51).
Report is 771 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #610      +/-   ##
==========================================
+ Coverage   35.78%   35.98%   +0.20%     
==========================================
  Files         189        7     -182     
  Lines       17570      678   -16892     
  Branches    11592      362   -11230     
==========================================
- Hits         6287      244    -6043     
+ Misses        994      134     -860     
+ Partials    10289      300    -9989     
Flag Coverage Δ
unittests 35.98% <ø> (+0.20%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 194 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bmagyar bmagyar merged commit 372725e into ros-controls:master May 22, 2023
@christophfroehlich christophfroehlich deleted the fix_includes branch June 25, 2023 18:01
mechwiz pushed a commit to mechwiz/ros2_controllers that referenced this pull request Jul 3, 2023
christophfroehlich added a commit to christophfroehlich/ros2_controllers that referenced this pull request Jul 25, 2023
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.

4 participants