← Back to issue list

plugin: plugin validation with only one part dep

View original Github issue

Metadata

Project
craft-parts
Number
#224
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #224 in commit 098bb5c5, limiting plugin validation to one part dependency and skipping verification for excluded nodes.

Resolution: closed resolved

Reason: PR #224 was approved and merged through commit 098bb5c5. Implementation commit 6eede329 added the requested validation behavior.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Setup plugin validation so it only mentions a dependency on one part. This is still not the best solution to the problem. Also fixup the node verification for it to check if node is to be included and if so, skip verification. - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- I am not entirely happy with this solution. Proposing for discussion.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #224 in commit 098bb5c5, limiting plugin validation to one part dependency and skipping verification for excluded nodes.
stealth/space-bunny-alpha — closed resolved Merged plugin validation changes to report only one part dependency and skip node verification when a node is not included.
qwen/qwen3.6-35b-a3b — — Merged after reviewer approval and passing CI. Updated plugin validation for single-part dependencies and fixed node verification to skip checks when nodes are included. The provisional solution was accepted for integration.
qwen3.6-35b-a3b-mtp-q6 — — Merged updates to plugin validation for single-part dependencies and corrected node verification to bypass checks when nodes are included. The maintainer accepted the workaround despite noting it as a temporary solution.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#197 plugins: validate dependencies craft-parts merged Merged PR #197 via ed3c56e7 (cf5769ad), consolidating plugin dependency validation ahead of conda, npm, and Rust plugins with no functional changes.
73%