Repository navigation
CI: build and test on ROS 2 Jazzy and Kilted - #9
Merged
Merged
Conversation
Rolling now targets Ubuntu 26.04, which has no ROS packages yet, and the old setup-ros versions use an expired ROS apt key. Build on the same distributions as the CoreSense Pixi channels, update the ros-tooling actions, and remove the scheduled trigger that made GitHub disable the workflow after 60 days of inactivity. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
No unresolved review comments remain, and the workflow updates are appropriately scoped.
Review effort: Lite
Findings: None
What changed in this PR
Updates ROS 2 CI to build and test Jazzy and Kilted on Ubuntu 24.04.
Changes:
- Replaces Rolling with Jazzy/Kilted matrix builds.
- Updates ROS tooling actions.
- Adds manual dispatch and removes obsolete scheduled/Codecov steps.
| File | Description |
|---|---|
.github/workflows/rolling.yaml |
Configures the updated ROS 2 matrix and CI actions. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
- Locate the templates from rossdl_cmake_DIR. The previous code looked for an install prefix containing 'rossdl_cmake', which only exists with colcon isolated installs, so it failed with merged installs, Debian and conda packages. - Always use the descriptions of the package being built as local files: they are visible to CMake from the build space but not installed yet, so the generator could not read them from the ament index. - Support EmPy 4 (conda-forge, pip) as well as EmPy 3 (Ubuntu). - Export the dependencies that the generators need at build time in the packages that use rossdl: ros2run, python3-yaml and ament_index_python. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The CI of this repository was failing or not running:
setup-rosversions use an expired ROS apt key.This PR:
ros-tooling/setup-rosto 0.7.19 andros-tooling/action-ros-cito 0.4.8;🤖 Generated with Claude Code