[Humble] Move the header files to .hpp extension (#206) - No deprecation notice

[saikishor]

No description provided.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 87.02703% with 24 lines in your changes missing coverage. Please review.

Project coverage is 77.28%. Comparing base (d3f9eef) to head (ac55a9f).

Files with missing lines	Patch %	Lines
...ude/realtime_tools/realtime_server_goal_handle.hpp	78.26%	3 Missing and 12 partials ⚠️
include/realtime_tools/realtime_buffer.hpp	86.53%	2 Missing and 5 partials ⚠️
include/realtime_tools/realtime_publisher.hpp	96.36%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           humble     #225   +/-   ##
=======================================
  Coverage   77.28%   77.28%           
=======================================
  Files           8        8           
  Lines         471      471           
  Branches      179      179           
=======================================
  Hits          364      364           
  Misses         70       70           
  Partials       37       37           

Flag	Coverage Δ
unittests	77.28% <87.02%> (ø)

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

Files with missing lines	Coverage Δ
include/realtime_tools/realtime_box.hpp	100.00% <100.00%> (ø)
src/realtime_clock.cpp	63.82% <ø> (ø)
include/realtime_tools/realtime_publisher.hpp	96.36% <96.36%> (ø)
include/realtime_tools/realtime_buffer.hpp	86.53% <86.53%> (ø)
...ude/realtime_tools/realtime_server_goal_handle.hpp	78.26% <78.26%> (ø)

[christophfroehlich]

Do we annoy users if we add a message (msvc, gcc but not sure for clang) instead of warnings like we do on newer distros? At least compilation would not break with -Werr then.

[saikishor]

Do we annoy users if we add a message (msvc, gcc but not sure for clang) instead of warnings like we do on newer distros? At least compilation would not break with -Werr then.

Sure! I can add a message instead of a warning

[saikishor]

Do we annoy users if we add a message (msvc, gcc but not sure for clang) instead of warnings like we do on newer distros? At least compilation would not break with -Werr then.

In file included from /home/user/rolling_worspace/ros2_control_ws/src/ros2_control/controller_manager/test/test_chainable_controller/test_chainable_controller.hpp:25,
                 from /home/user/rolling_worspace/ros2_control_ws/src/ros2_control/controller_manager/test/test_controller_manager_srvs.cpp:30:
/home/user/rolling_worspace/ros2_control_ws/src/realtime_tools/include/realtime_tools/realtime_buffer.h:22:100: note: ‘#pragma message: This header include is deprecated. Please update your code to use 'realtime_buffer.hpp' header.’
   22 |   "This header include is deprecated. Please update your code to use 'realtime_buffer.hpp' header.")  //NOLINT