Skip to content

Add warning that actions require ament_cmake, not ament_python #4986 #5201

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 13 commits into
base: rolling
Choose a base branch
from
+12 −2
Open

Add warning that actions require ament_cmake, not ament_python #4986 #5201

Changes from 2 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
fcb450c
Add warning that actions require ament_cmake, not ament_python #4986
vimal0athithan Mar 30, 2025
821f030
Applied reviewer suggestions: Moved lines 45-46 outside warning block…
vimal0athithan Apr 12, 2025
ccdb40e
Fix linting and test failures based on check logs
vimal0athithan Apr 16, 2025
be69146
Update source/Tutorials/Intermediate/Creating-an-Action.rst
kscottz Apr 17, 2025
bc04c8f
Update source/Tutorials/Intermediate/Creating-an-Action.rst
kscottz Apr 17, 2025
87be1e0
Update source/Tutorials/Intermediate/Creating-an-Action.rst
kscottz Apr 17, 2025
95542c1
Update source/Tutorials/Intermediate/Creating-an-Action.rst
kscottz Apr 17, 2025
054e030
Fix trailing whitespace on line 44 and add newline at end of file
vimal0athithan Apr 18, 2025
55215ad
Fix trailing whitespace on line 44 and add newline at end of file
vimal0athithan Apr 18, 2025
f362035
Update source/Tutorials/Intermediate/Creating-an-Action.rst
kscottz Apr 28, 2025
8b2178c
Update source/Tutorials/Intermediate/Creating-an-Action.rst
kscottz Apr 28, 2025
2f1b4e6
Update source/Tutorials/Intermediate/Creating-an-Action.rst
kscottz Apr 28, 2025
ddd6e82
Update source/Tutorials/Intermediate/Creating-an-Action.rst
kscottz Apr 28, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 9 additions & 1 deletion source/Tutorials/Intermediate/Creating-an-Action.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,14 @@ Tasks
1 Creating an interface package
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Actions can only be created in a C++ package using ``ament_cmake``. They are not supported
Copy link
Collaborator

@fujitatomoya fujitatomoya Apr 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not think this is true. it does not have to be C++ package, rather it has to be CMake package.

in Python packages using ``ament_python``. This restriction also applies to messages and services, so
ensure your package is configured as a C++ package before proceeding with this tutorial.

.. note::
It is recommended to define action interfaces in this tutorial. This improves modularity and makes it
easier to reuse them across multiple projects.

.. tabs::

.. group-tab:: Linux
Expand Down Expand Up @@ -211,4 +219,4 @@ Next, let's utilize your newly defined action interface by creating an action se
Related content
---------------

For more detailed information about ROS actions, please refer to the `design article <http://design.ros2.org/articles/actions.html>`__.
For more detailed information about ROS actions, please refer to the `design article <http://design.ros2.org/articles/actions.html>`__.
Loading