Skip to content

Update action python demos to support setting introspection#708

Merged
fujitatomoya merged 2 commits intoros2:rollingfrom
Barry-Xu-2018:review/topic-action-introspection-py-demo
Mar 14, 2025
Merged

Update action python demos to support setting introspection#708
fujitatomoya merged 2 commits intoros2:rollingfrom
Barry-Xu-2018:review/topic-action-introspection-py-demo

Conversation

@Barry-Xu-2018
Copy link
Contributor

@Barry-Xu-2018 Barry-Xu-2018 commented Feb 17, 2025

Signed-off-by: Barry Xu <barry.xu@sony.com>
@Barry-Xu-2018 Barry-Xu-2018 marked this pull request as ready for review March 13, 2025 01:23
@fujitatomoya fujitatomoya self-requested a review March 13, 2025 04:37
Copy link
Collaborator

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

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

lgtm with green CI.

Copy link
Collaborator

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

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

one comment needs to be fixed.

@fujitatomoya
Copy link
Collaborator

@Barry-Xu-2018 the fix required only for doc section, i will go ahead to start CI.

@fujitatomoya
Copy link
Collaborator

Pulls: #708
Gist: https://gist.githubusercontent.com/fujitatomoya/3c109b89077a5b5737d10ef79b4af665/raw/566663393370a4b66116ee8a322e72ea76c60f2b/ros2.repos
BUILD args: --packages-above-and-dependencies action_tutorials_py
TEST args: --packages-above action_tutorials_py
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/15378

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

Signed-off-by: Barry Xu <barry.xu@sony.com>
@fujitatomoya fujitatomoya merged commit 61e8dd4 into ros2:rolling Mar 14, 2025
2 checks passed
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.

2 participants