Skip to content

Add support check for content filter feature in subscription#3089

Merged
fujitatomoya merged 1 commit intoros2:rollingfrom
Barry-Xu-2018:develop/topic-use-content-filter-support-check
Mar 17, 2026
Merged

Add support check for content filter feature in subscription#3089
fujitatomoya merged 1 commit intoros2:rollingfrom
Barry-Xu-2018:develop/topic-use-content-filter-support-check

Conversation

@Barry-Xu-2018
Copy link
Copy Markdown
Collaborator

Description

Add support check interface for content filter feature in subscription.
Based on this new interface, optimize the current code and related test code.

Depend on ros2/rcl#1293

Is this user-facing behavior change?

No.

Did you use Generative AI?

No.

Additional Information

Signed-off-by: Barry Xu <barry.xu@sony.com>
@fujitatomoya fujitatomoya marked this pull request as ready for review March 10, 2026 23:13
Copy link
Copy Markdown
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.

@fujitatomoya
Copy link
Copy Markdown
Collaborator

Pulls: #3089
Gist: https://gist.githubusercontent.com/fujitatomoya/89b99d2bedf822ce27f62859a001ec19/raw/7ea982b69267b22563b58d5caa75c7053ed17247/ros2.repos
BUILD args: --packages-above-and-dependencies rclcpp
TEST args: --packages-above rclcpp
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/18431

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

@Barry-Xu-2018
Copy link
Copy Markdown
Collaborator Author

Barry-Xu-2018 commented Mar 11, 2026

Re-run CI due to CI internal issues.

  • Linux Build Status
  • Linux-aarch64 Build Status

@fujitatomoya fujitatomoya requested a review from mjcarroll March 12, 2026 02:38
@fujitatomoya fujitatomoya merged commit df2ac88 into ros2:rolling Mar 17, 2026
2 of 3 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.

3 participants