Rename class type to plugin name #api-breaking #abi-breaking

[bmagyar]

Changes to internal API, not really an issue but I think that the message changes justify API and ABI break flags

Closes #624

[codecov-commenter]

Codecov Report

Merging #780 (b1bbf8d) into master (925f5f3) will decrease coverage by 1.71%.
The diff coverage is 37.18%.

@@            Coverage Diff             @@
##           master     #780      +/-   ##
==========================================
- Coverage   34.61%   32.90%   -1.72%     
==========================================
  Files          52       90      +38     
  Lines        2981     8510    +5529     
  Branches     1855     5668    +3813     
==========================================
+ Hits         1032     2800    +1768     
- Misses        310      674     +364     
- Partials     1639     5036    +3397     

Flag	Coverage Δ
unittests	32.90% <37.18%> (-1.72%)	⬇️

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

Impacted Files	Coverage Δ
controller_manager/src/realtime.cpp	0.00% <0.00%> (ø)
controller_manager/src/ros2_control_node.cpp	0.00% <0.00%> (ø)
..._interface/include/hardware_interface/actuator.hpp	100.00% <ø> (ø)
...re_interface/include/hardware_interface/sensor.hpp	100.00% <ø> (ø)
...re_interface/include/hardware_interface/system.hpp	100.00% <ø> (ø)
hardware_interface/src/resource_manager.cpp	50.09% <ø> (-3.53%)	⬇️
hardware_interface/src/sensor.cpp	50.52% <ø> (ø)
hardware_interface/src/system.cpp	55.45% <ø> (ø)
...rface/test/mock_components/test_generic_system.cpp	8.62% <ø> (ø)
...dware_interface/test/test_component_interfaces.cpp	32.44% <ø> (+4.25%)	⬆️
... and 106 more

[destogl]

Fully agree, but let's deprecate it first.

[destogl]

Just a small comments to remove additional two lines before merging