Fix a few licenses in ros-related test files
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Here is the follow-up PR to https://github.com/snapcore/snapcraft/pull/3405 fixing a few licenses in ROS-related test files. The license declared in a few `package.xml` did not match those in sources.
See [this discussion](https://github.com/snapcore/snapcraft/pull/3405#discussion_r554162049) for reference.
Signed-off-by: artivis <deray.jeremie@gmail.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a follow-up to PR #3405 correcting license declarations in ROS-related test files. The update aligned package.xml metadata with actual source code licenses, resolving compliance mismatches in the test suite. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a follow-up to PR #3405 correcting license mismatches between package.xml declarations and source code in ROS-related test files. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5957 docs: fix typo in ROS version names | snapcraft | merged | Merged a documentation update correcting a typo in ROS version names. Approved by two reviewers, modified one file, and passed all CI checks. A reviewer confirmed ReadTheDocs was misreporting the build status. | |
| #5465 docs: fix typos and format ROS guides | snapcraft | merged | Merged documentation update fixing typos and formatting ROS guides. Approved by two reviewers and passed CI checks. Changes were limited to source formatting and typo corrections across two files with no content modifications. | |
| #5232 docs: migrate ros 1 extensions | snapcraft | merged | Merged documentation migration for ROS 1 extensions. Approved by two reviewers, passed CI and tests, and maintained coverage. Updates six files with 307 additions and 10 deletions. | |
| #4828 Add ROS 2 Jazzy content-sharing support | snapcraft | merged | 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. | |
| #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. | |
| #515 Clean up the ROS example test. | snapcraft | merged | Merged a cleanup of the ROS example test by maintainer kyrofa. Modified two files with 36 additions and 24 deletions. Accepted and merged without reviewer comments or CI checks. |