Add doc-string warnings for destroy methods for services. (#1205)

Signed-off-by: Tomoya Fujita <[email protected]>

rclpy/rclpy/client.py

@@ -217,4 +217,10 @@ def service_name(self) -> str:
             return self.__client.service_name
 
     def destroy(self):
+        """
+        Destroy a container for a ROS service client.
+
+        .. warning:: Users should not destroy a service client with this destructor, instead they
+           should call :meth:`.Node.destroy_client`.
+        """
         self.__client.destroy_when_not_in_use()

rclpy/rclpy/publisher.py

@@ -38,7 +38,7 @@ def __init__(
         """
         Create a container for a ROS publisher.
 
-        .. warning:: Users should not create a publisher with this constuctor, instead they should
+        .. warning:: Users should not create a publisher with this constructor, instead they should
            call :meth:`.Node.create_publisher`.
 
         A publisher is used as a primary means of communication in a ROS system by publishing

rclpy/rclpy/service.py

@@ -40,7 +40,7 @@ def __init__(
         """
         Create a container for a ROS service server.
 
-        .. warning:: Users should not create a service server with this constuctor, instead they
+        .. warning:: Users should not create a service server with this constructor, instead they
            should call :meth:`.Node.create_service`.
 
         :param service_impl: :class:`_rclpy.Service` wrapping the underlying ``rcl_service_t``
@@ -108,4 +108,10 @@ def service_name(self) -> str:
             return self.__service.name
 
     def destroy(self):
+        """
+        Destroy a container for a ROS service server.
+
+        .. warning:: Users should not destroy a service server with this destructor, instead they
+           should call :meth:`.Node.destroy_service`.
+        """
         self.__service.destroy_when_not_in_use()