v2 plugins: add catkin plugin
Metadata
Current evaluation
Merged after passing review and CI checks. Introduced a new Catkin plugin using the v2 plugin API and refactored shared code from the Colcon plugin. Additional spread tests were added to verify all plugin options.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh tests/unit`?
-----
Add a new Catkin plugin using v2 of the plugin API. Refactor common functions out of the Colcon plugin to share functionality with both.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after passing review and CI checks. Introduced a new Catkin plugin using the v2 plugin API and refactored shared code from the Colcon plugin. Additional spread tests were added to verify all plugin options. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a new v2 Catkin plugin, refactoring shared logic from the Colcon plugin. Comprehensive spread tests were added to cover all options, and runtime constraints were adjusted to finalize the integration. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Added a new Catkin plugin using the v2 plugin API and refactored shared code from the Colcon plugin. Spread tests were added to verify all options and execution paths. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3282 v2 plugins: add catkin-tools plugin | snapcraft | merged | Merged. Introduces a catkin-tools plugin using the v2 plugin API, reusing shared ROS functionality from existing Catkin and Colcon plugins. Passed static and unit tests, approved by two reviewers, and successfully integrated. | |
| #3037 plugins: introduce v2.CMakePlugin | snapcraft | merged | Merged introduction of v2.CMakePlugin and consolidation of the v2 plugin spread test suite to streamline future development. Approved by one reviewer, passed CI, and modified 16 files with 196 additions and 23 deletions. | |
| #3664 ROS plugins v2: respect source-subdir key | snapcraft | merged | Merged. Updates ROS plugins v2 to respect the source-subdir key, removes an unnecessary catkin_tools dependency, and adds tests for Colcon and catkin v2 subdir handling. Approved by reviewers and passed CI. | |
| #3566 ROS 2 plugins v2 misc fixes | snapcraft | merged | Merged after resolving conflicts and passing CI. Fixes ROS 2 plugins v2 by updating the colcon install path, resolving local packages during staging by removing the COLCON_IGNORE marker, and adding spread tests. | |
| #2456 plugins: add colcon plugin | snapcraft | merged | Merged the colcon plugin for ROS2 builds, resolving LP: #1805213. Added as a beta feature, the change was approved by two reviewers, passed CI with 100% diff coverage, and introduced 2120 lines across 16 files. | |
| #2206 tests: add spread suite for catkin plugin | snapcraft | merged | Merged migration of Catkin plugin integration tests to a Spread suite, resolving Launchpad bug #1783828. Approved by one reviewer, passed CI, and maintained coverage across 37 files. | |
| #1593 catkin tools plugin: add catkin tools support. | snapcraft | merged | Merged a new catkin tools plugin replacing catkin_make. After review, conflict resolution, and adding a beta warning, the changes were approved. Brief testing infrastructure delays preceded successful integration. | |
| #1001 plugins: update catkin plugin to use get_pull_properties() | snapcraft | merged | Merged changes update the catkin plugin to use get_pull_properties() and add corresponding unit tests. Approved by one reviewer, passed CI checks, and improved overall code coverage. The merge modified two files with a net addition of four lines. |