Skip to content

Add ROS2 Jazzy build and test automation - #8

Merged
yassinsolim merged 8 commits into
mainfrom
chore/ros2-build-ci
Jul 25, 2026
Merged

Add ROS2 Jazzy build and test automation#8
yassinsolim merged 8 commits into
mainfrom
chore/ros2-build-ci

Conversation

@Haroldmin1028

Copy link
Copy Markdown
Contributor

This PR adds a GitHub Actions workflow to automate the build and test verification of the WayBionic workspace.

Changes

  • Added a package-aware workflow ros2_build_test.yml running on a standard ros:jazzy container
  • Replaced explicit folder name references with dynamically resolved folder names

Commands replicated by automation:

rosdep install --from-paths src --ignore-src -r -y
colcon build --packages-select $PACKAGES
colcon test --packages-select $PACKAGES
colcon test-result --all

Verification logs:

build/waybionic_bringup/Testing/20260717-2007/Test.xml: 4 tests, 0 errors, 0 failures, 0 skipped
build/waybionic_bringup/test_results/waybionic_bringup/flake8.xunit.xml: 1 test, 0 errors, 0 failures, 0 skipped
build/waybionic_bringup/test_results/waybionic_bringup/lint_cmake.xunit.xml: 1 test, 0 errors, 0 failures, 0 skipped
build/waybionic_bringup/test_results/waybionic_bringup/pep257.xunit.xml: 1 test, 0 errors, 0 failures, 0 skipped
build/waybionic_bringup/test_results/waybionic_bringup/xmllint.xunit.xml: 1 test, 0 errors, 0 failures, 0 skipped
build/waybionic_description/Testing/20260717-2007/Test.xml: 2 tests, 0 errors, 0 failures, 0 skipped
build/waybionic_description/test_results/waybionic_description/lint_cmake.xunit.xml: 1 test, 0 errors, 0 failures, 0 skipped
build/waybionic_description/test_results/waybionic_description/xmllint.xunit.xml: 1 test, 0 errors, 0 failures, 0 skipped

@yassinsolim yassinsolim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The workflow runs successfully, but it does not test the entire current workspace. waybionic_rviz_plugins is already on main, while PACKAGES only includes waybionic_description and waybionic_bringup.

Please remove the package allowlist and run colcon build, colcon test, and colcon test-result --all against the full workspace. That will automatically cover current and future packages without manually updating the workflow. Once that is fixed, this is good to merge.

yassinsolim
yassinsolim previously approved these changes Jul 24, 2026

@yassinsolim yassinsolim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. The package allowlist has been removed, so the workflow now builds and tests the full workspace, including waybionic_rviz_plugins.
I verified the merge result locally, and the actual Ubuntu GitHub Actions run passed dependency installation, the full build, and all tests. This is good to merge.

@Haroldmin1028
Haroldmin1028 dismissed yassinsolim’s stale review July 24, 2026 23:13

The merge-base changed after approval.

yassinsolim
yassinsolim previously approved these changes Jul 24, 2026

@yassinsolim yassinsolim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

@Haroldmin1028
Haroldmin1028 dismissed yassinsolim’s stale review July 24, 2026 23:20

The merge-base changed after approval.

@yassinsolim
yassinsolim merged commit e963d5b into main Jul 25, 2026
1 check 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