Skip to content

remove default: so that compiler can detect the missing case.#3048

Merged
ahcorde merged 1 commit intorollingfrom
fujitatomoya/remove-default-with-enum
Feb 2, 2026
Merged

remove default: so that compiler can detect the missing case.#3048
ahcorde merged 1 commit intorollingfrom
fujitatomoya/remove-default-with-enum

Conversation

@fujitatomoya
Copy link
Copy Markdown
Collaborator

Description

cosmetic minor fixes.

Fixes # (issue)

Is this user-facing behavior change?

Did you use Generative AI?

Additional Information

Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
@fujitatomoya
Copy link
Copy Markdown
Collaborator Author

fujitatomoya commented Jan 31, 2026

Pulls: #3048
Gist: https://gist.githubusercontent.com/fujitatomoya/6240d148a2d346a69b97122647af25d5/raw/e14b0665565547676d057570ec8f1043b7e4c491/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/18079

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

@ahcorde ahcorde merged commit b009212 into rolling Feb 2, 2026
2 of 3 checks passed
@ahcorde ahcorde deleted the fujitatomoya/remove-default-with-enum branch February 2, 2026 11:31
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