← Back to issue list

legacy/plugins: improve nil plugin docstring

View original Github issue

Metadata

Project
snapcraft
Number
#3949
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged as 64d54ddfb, improving the legacy nil plugin docstring; it remade the earlier unmerged PR #3913.

Resolution: closed resolved

Reason: PR #3949 was approved and merged into main as commit 64d54ddfb, adding useful documentation for the legacy nil plugin.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Update nil plugin documentation with useful information. This is a remake of PR #3913 that was closed before merging. Co-authored-by: Robert P. J. Day <robert.day@canonical.com> Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com> - [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/)? - [ ] Have you successfully run `make lint`? - [ ] Have you successfully run `pytest tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged as 64d54ddfb, improving the legacy nil plugin docstring; it remade the earlier unmerged PR #3913.
stealth/space-bunny-alpha — closed resolved Merged as commit 64d54ddfb, improving the legacy nil plugin docstring with useful documentation; it replaced the earlier closed PR #3913.
stealth/space-bunny-alpha — closed resolved Merged into main as commit 64d54ddfb, improving the legacy nil plugin docstring. PR #3949 remade the earlier unmerged PR #3913.
qwen/qwen3.6-35b-a3b — — 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.
qwen3.6-35b-a3b-mtp-q6 — — Merged into main. Updates the nil plugin docstring with improved documentation, replacing the previously closed PR 3913. Codecov reports a minor 0.08% coverage decrease.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft#3913 (confidence 100%)

    Earlier PR was closed before merging; #3949 remade its documentation changes.

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.
84%
#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.
71%
#330 plugins: Correct docstring in Dotnet plugin craft-parts merged Merged PR #330 as commit a50c72b3, correcting a one-line copy-and-paste docstring typo in the Dotnet plugin.
70%