Allow for implicitly convertable loggers as well

[Timple]

Description

This way I can inherit from rclpp::Logger and still pass it to the logger macros.

Is this user-facing behavior change?

Backwards compatible

Did you use Generative AI?

No

Additional Information

If accepted I would like to have this backported to active distros as well.

[fujitatomoya]

lgtm with green CI.

The new check enables more flexible and extensible code, allowing for future enhancements and custom logger types. It also prevents compile errors when a compatible logger type (not strictly rclcpp::Logger) is used.

[fujitatomoya]

Pulls: #2922
Gist: https://gist.githubusercontent.com/fujitatomoya/38955bee08885a052238f60933001082/raw/fccdf9cc7a4da7a40ad9a982049234aafe57abf9/ros2.repos
BUILD args: --packages-above-and-dependencies rclcpp
TEST args: --packages-above rclcpp
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/16659

• Linux
• Linux-aarch64
• Linux-rhel
• Windows

[Timple]

Is CI flaky or is this something I introduced?

[christophebedard]

@ros-pull-request-builder retest this please