Test supported ROS distributions - #45
Conversation
rlpratt12
left a comment
There was a problem hiding this comment.
Why are we deleting rolling and not just adding lyrical in addition?
a62bc9a to
7b50904
Compare
|
[written by AI] Addressed the review: Rolling coverage is retained, but moved from The matrix is now Humble/main, Jazzy/main, Lyrical/main, and Rolling/testing. I also fixed the Lyrical compile failures exposed by the new row and verified the full package tests in clean Humble, Jazzy, and Lyrical containers. |
7b50904 to
84adf5a
Compare
|
[written by AI] The previous green run exposed a branch-policy compatibility issue: adding This head explicitly names each real matrix job |
[written by AI]
Problem
The existing Rolling/main CI row maps to Ubuntu Resolute, where the ROS main repository does not publish
ros-rolling-ros-environment. Lyrical/main is installable, but compiling this repository on Lyrical also exposed removedament_cmakeandhardware_interfaceAPIs.Fix
ament_target_dependencies()calls with imported CMake targets.HardwareInfoinitialization API and theHardwareComponentInterfaceParamsAPI introduced beforehardware_interface6 and required from 6.0 onward.contents: readpermissions.This keeps support-matrix and compatibility maintenance separate from the SPDX-only change in #44.
Validation
actionlinton the changed workflowgit diff --check7b5090461140ed79fa3c34d8539317a69c96ed6b