Skip to content

Improve CI triggers and add ROS 2 C++ node validation#171

Merged
jonyMarino merged 5 commits into
mainfrom
infra/enhace_github_actions
Jul 15, 2026
Merged

Improve CI triggers and add ROS 2 C++ node validation#171
jonyMarino merged 5 commits into
mainfrom
infra/enhace_github_actions

Conversation

@jonyMarino

Copy link
Copy Markdown
Contributor

Fixes: N/A

About

This PR improves the GitHub Actions workflows by limiting each workflow to relevant source changes and adding CI coverage for the ROS 2 C++ node.

ClickUp task: https://app.clickup.com/t/90131502978/86ajh9j92

Main changes:

  • Add a ROS 2 Humble job that builds projectairsim_ros2_cpp and runs its unit tests.
  • Integrate the ROS 2 job into the existing C++ client workflow.
  • Detect C++ client and ROS 2 changes independently to avoid unnecessary jobs.
  • Run the C++ client workflow for its actual core_sim and packaging dependencies.
  • Restrict the documentation workflow to documentation and Python autodoc changes.
  • Restrict Windows and Linux SimLibs workflows to relevant components.
  • Restrict the self-hosted integration workflow to simulator, Unreal, and Python client changes.
  • Rename the Linux SimLibs workflow from Debug to Release to match the targets it executes.
  • Run Linux SimLibs Release tests on Ubuntu 22.04.
  • Use setup_linux_dev_tools.sh to prepare the Linux SimLibs environment.
  • Preserve manual workflow execution through workflow_dispatch.

How Has This Been Tested?

  • Parsed all modified workflow files successfully as YAML.
  • Ran git diff --check successfully.
  • Verified workflow path filters against the build scripts and CMake dependencies.
  • Verified that the ROS 2 job targets ROS 2 Humble on Ubuntu 22.04 and reports unit test failures through colcon.
  • Verified that the Linux SimLibs workflow uses the repository development setup script.

The workflows themselves require GitHub-hosted or self-hosted runner environments and were not executed locally.

Screenshots and videos (if appropriate):

N/A

@jonyMarino
jonyMarino merged commit 7fb9c56 into main Jul 15, 2026
8 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.

1 participant