← Back to issue list

catkin plugin: describe how to build all packages

View original Github issue

Metadata

Project
snapcraft
Number
#2459
Type
pull request
State
merged
Author
kyrofa
Labels
Created
Updated
Closed

Current evaluation

Merged a documentation update for the catkin plugin explaining how to build all packages. Adds a missing blurb from a previous bug fix. Approved by two reviewers, passed CI, and modified one file.

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] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh tests/unit`? ----- Add the documentation blurb left out of the fix for [LP: #1721168.](https://bugs.launchpad.net/snapcraft/+bug/1721168).

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged a documentation update for the catkin plugin explaining how to build all packages. Adds a missing blurb from a previous bug fix. Approved by two reviewers, passed CI, and modified one file.
qwen3.6-35b-a3b-mtp-q6 Merged documentation update for the catkin plugin detailing how to build all packages. Completes the fix for LP: #1721168 by adding previously omitted instructions. No code changes were made.
qwen3.6-35b-a3b-mtp-q6 Merged documentation update for the catkin plugin explaining how to build all packages. This change completes a previous fix for LP: #1721168 by adding the missing documentation blurb.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5729 docs: migrate 'the catkin-tools plugin' snapcraft merged Merged documentation update migrating the catkin-tools plugin. Approved by two reviewers with all CI checks passing. Changes span three files, adding 87 lines and removing 5.
77%
#5718 docs: migrate the catkin plugin reference snapcraft merged Merged documentation update migrating the catkin plugin reference. Approved by three reviewers and passed most CI checks despite one failing integration test. Ready for merge after hotfix branch targeting discussion.
74%
#175 Catkin plugin: Refactor build. snapcraft merged Merged after addressing reviewer feedback on error handling and testing strategy. The update refactored the Catkin plugin build process, simplified build logic, extracted dependency resolution, added a source-space keyword, and achieved 100% unit test coverage.
73%
#1779 catkin-tools plugin: use stage-packages snapcraft merged Merged a fix switching the catkin-tools plugin from build-packages to stage-packages, resolving clean system failures. Added an integration test. Approved by two reviewers and passed CI.
73%
#2545 catkin plugin: use build-packages for compilers snapcraft merged Merged updates to the Catkin plugin that specify compilers as build-packages instead of isolating them. This resolves a rebuild failure and aligns with current base requirements. Approved by one reviewer and passed CI.
72%
#1743 catkin plugin: support building entire workspace snapcraft merged Merged. The catkin-packages property is now optional. When omitted, the plugin builds the entire workspace. Resolves LP: #1721168. Approved by two reviewers with passing CI checks.
72%
#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.
71%
#71 catkin plugin: typo s/packages.xml/package.xml/ snapcraft merged Merged a typo fix in the catkin plugin, correcting packages.xml to package.xml across two files. Approved by @ted-gould and merged by maintainer sergiusens.
70%