← Back to issue list

Nil plugin: Make purpose more clear in documentation.

View original Github issue

Metadata

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

Current evaluation

Merged in snapcraft#455 (391143bc1), clarifying that nil cannot pull sources, adding examples and tests, and rejecting unsupported source keywords.

Resolution: closed resolved

Reason: PR #455 was merged as commit 391143bc1. It clarified the nil plugin's purpose and added validation and tests preventing use of unsupported source keywords.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Some users were under the impression that nil could pull sources, when it can't. This PR resolves LP: [#1549691](https://bugs.launchpad.net/snapcraft/+bug/1549691) by hopefully making that more clear, and giving an example of what it CAN do.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged in snapcraft#455 (391143bc1), clarifying that nil cannot pull sources, adding examples and tests, and rejecting unsupported source keywords.
stealth/space-bunny-alpha — closed resolved Merged in canonical/snapcraft#455 (commit 391143bc1), clarifying that the nil plugin cannot pull sources, adding examples and tests, and rejecting unsupported source keywords.
qwen/qwen3.6-35b-a3b — — 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.
qwen3.6-35b-a3b-mtp-q6 — — Merged PR clarifying the nil plugin cannot pull sources, resolving LP #1549691. Added validation to reject invalid keywords like source, plus integration and unit tests to enforce the restriction.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#300 plugins: improve nil plugin docstring craft-parts merged Merged PR #300 in commit d7c1b68d, expanding the nil plugin docstring with guidance for source-less parts and parts using override-build.
74%
#85 Add nil plugin snapcraft merged Merged snapcraft#85 via 8e62b401f, adding a no-op nil plugin for parts whose properties are all optional and incorporating integration tests.
72%
#1549691 Improve documentation for the nil plugin snapcraft (launchpad) closed Documentation clarified by PR #455, merged as commit 391143bc1. Requested source support was rejected because sources are private per part and left to the copy plugin.
72%