Skip to content

Use InvalidServiceTypeError for unavailable service type in GenericClient#2629

Merged
fujitatomoya merged 1 commit intoros2:rollingfrom
Barry-Xu-2018:topic-update-exception-generic-client
Sep 16, 2024
Merged

Use InvalidServiceTypeError for unavailable service type in GenericClient#2629
fujitatomoya merged 1 commit intoros2:rollingfrom
Barry-Xu-2018:topic-update-exception-generic-client

Conversation

@Barry-Xu-2018
Copy link
Copy Markdown
Collaborator

Use newly defined exceptions rclcpp::exceptions::InvalidServiceTypeError to make errors clearer.
rclcpp::exceptions::InvalidServiceTypeError was introduced in #2617.

…ient

Signed-off-by: Barry Xu <barry.xu@sony.com>
@ahcorde
Copy link
Copy Markdown
Contributor

ahcorde commented Sep 16, 2024

Pulls: #2629
Gist: https://gist.githubusercontent.com/ahcorde/26b4db9149a95e898725c9c16e0eb14a/raw/3c47a4a32f8b42d2648901e51f2933c6b3b46e4f/ros2.repos
BUILD args: --packages-above-and-dependencies rclcpp --packages-above-and-dependencies rclcpp
TEST args: --packages-above rclcpp --packages-above rclcpp
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/14564

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@fujitatomoya fujitatomoya merged commit 918363d into ros2:rolling Sep 16, 2024
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.

3 participants