plugins: add the nil plugin
Metadata
Current evaluation
Merged the nil plugin to handle parts with no source. Approved by two reviewers, passed CI checks, and introduced 109 lines across two files.
Suggested action: —
No scores available.
Issue body
The nil plugin is used to handle parts with no source.
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged the nil plugin to handle parts with no source. Approved by two reviewers, passed CI checks, and introduced 109 lines across two files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged the nil plugin to handle parts with no source. Submitted by maintainer cmatsuoka, the change was accepted and integrated without discussion. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #300 plugins: improve nil plugin docstring | craft-parts | merged | Merged following approval and passing CI checks. The update revised the nil plugin docstring with additional documentation details, modifying one file. | |
| #85 Add nil plugin | snapcraft | merged | The nil plugin for parts with all optional properties was merged after addressing reviewer requests to remove the icon and add an integration test, followed by conflict resolution. | |
| #1518404 Add a nil plugin | snapcraft (launchpad) | closed | Closed without implementation. The feature request to add a nil plugin for parts using only built-in keywords was abandoned due to lack of discussion or follow-up work. | |
| #703 feat(docs): nil plugin reference and howto | craft-parts | merged | Merged documentation update adding nil plugin reference and how-to guide. Approved by two reviewers, modifying three files with 50 additions. Addresses CRAFT-2682. | |
| #2201 tests: add spread suite for nil plugin | snapcraft | merged | Merged to add a Spread test suite for Nil plugin integration tests, resolving Launchpad bug #1783838. Approved by reviewers, passed CI, and updated 8 files. | |
| #3949 legacy/plugins: improve nil plugin docstring | snapcraft | merged | Merged into main to update the nil plugin docstring in legacy/plugins, replacing the closed PR #3913. The change adds documentation details, passed CI, received approval, and was integrated with a negligible coverage decrease. | |
| #1610 Make the `nil` plugin out-of-source | craft-parts | closed | The nil plugin out-of-source change was abandoned. The author closed the pull request without merging following discussions in issue #1609, indicating the proposed breaking change was not adopted. | |
| #455 Nil plugin: Make purpose more clear in documentation. | snapcraft | merged | Merged documentation updates clarifying the nil plugin cannot pull sources, resolving user confusion. Added integration and unit tests to enforce the source restriction and validate schema changes. | |
| #3022 plugins: introduce v2.PluginV2 and v2.NilPlugin | snapcraft | merged | Merged after introducing v2.PluginV2 and v2.NilPlugin classes. Extended PluginHandler to support the new plugin version, added required attributes, and removed obsolete state file migration code. Author resolved initial CI failures prior to merge. | |
| #2326 plugins: remove implicit source | snapcraft | merged | Merged. Removes implicit source from plugins, resolving Launchpad bug #1776807. Approved, passed CI, and refined internal logic from PR #2300. Updated 46 files (+84/-59 lines). |