I would like to point out that identifiers like “RCLCPP_ACTION__QOS_HPP_” and “RCLCPP__EXECUTOR_HPP_” do not fit to the expected naming convention of the C++ language standard.
Would you like to adjust your selection for unique names?