← Back to issue list

Add ROS 2 Jazzy content-sharing support

View original Github issue

Metadata

Project
snapcraft
Number
#4828
Type
pull request
State
merged
Author
artivis
Labels
Created
Updated
Closed

Current evaluation

Added ROS 2 Jazzy content-sharing support. Merged after approval by two reviewers and passing CI checks. Changes span 13 files (+627/-18). Author noted unrelated failing spread tests.

Suggested action:

No scores available.

Issue body

Follows #4791 - [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 `tox run -m lint`? - [x] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`) -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Added ROS 2 Jazzy content-sharing support. Merged after approval by two reviewers and passing CI checks. Changes span 13 files (+627/-18). Author noted unrelated failing spread tests.
qwen3.6-35b-a3b-mtp-q6 Merged changes adding ROS 2 Jazzy content-sharing support. Follows #4791 and passed lint and test checks. Two unrelated spread test failures were noted but did not block integration.
qwen3.6-35b-a3b-mtp-q6 Merged after author clarification regarding unrelated failing spread tests. The update adds ROS 2 Jazzy content-sharing support, following up on issue #4791.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#4940 ci: enable jazzy content sharing arm64 spread test snapcraft merged Merged after two approvals. Enables ROS 2 Jazzy content-sharing spread tests for arm64. CI passed. An unrelated spread failure was noted but did not block the merge.
80%
#4791 extension: add ROS 2 Jazzy (core24) snapcraft merged Merged the ROS 2 Jazzy extension for core24. Approved by reviewers, the change modifies 13 files with 722 additions and 20 deletions. CI passed after updating codespell dependencies to resolve legacy flagging.
78%
#6234 docs(ros): update demo launch file used for jazzy snapcraft merged Merged a documentation fix updating the ROS 2 jazzy app craft tutorial to reference the correct demo launch file. Approved by three reviewers and integrated after passing CI checks.
74%
#1186 demos: add ROS content sharing demo snapcraft merged Merged changes adding a ROS content sharing demo snap and automating the standalone ROS demo test, resolving Launchpad bug #1664776. Approved by one reviewer, modified 15 files, building upon PR #1140.
73%
#3536 ROS plugins v2 misc fixes snapcraft merged Merged. Fixes ROS plugins v2 staging failures by checking local ROS directories and skipping rosdep resolution for locally available packages. Updated tests for core20. Approved by two reviewers and passed CI.
72%
#5227 docs: migrate ros 2 extensions snapcraft merged Merged documentation migration for ROS 2 extensions. Added eight files for foxy, humble, and jazzy. Passed CI and coverage checks. Reviewers approved the structure, and the maintainer merged the changes after consolidating the main ROS 2 pages.
72%
#3409 Fix a few licenses in ros-related test files snapcraft merged Merged to fix license mismatches in ROS-related test files by aligning package.xml declarations with source code. Approved by two reviewers, passed CI, and updated 12 files.
71%
#4223 ros: make linter tests more flexible snapcraft merged Merged changes to make ROS linter tests more flexible, resolving flaky failures in the linter-ros2-humble-mixed spread test caused by fluctuating unused library lists. Approved and passed CI.
71%
#6139 fix: ros content sharing non upstream package 8 14 snapcraft merged Merged a fix updating the ROS 2 plugin to handle non-upstream package dependencies and replace deprecated rospack with ros2 pkg. The changes supersede PR #6120, incorporate requested unit tests, and passed review and CI validation.
70%