← Back to issue list

lifecycle: add part validator (CRAFT-367)

View original Github issue

Metadata

Project
craft-parts
Number
#93
Type
pull request
State
closed
Author
cmatsuoka
Labels
Created
Updated
Closed

Current evaluation

Superseded without merging. The validator was simplified and moved to the application layer instead of being added to craft-parts.

Suggested action:

No scores available.

Issue body

Validate the given part data against the common parts model and the part's plugin model before creating the lifecycle manager. This allows the validation to happen on the host while the lifecycle manager is created later in a managed environment. 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 Superseded without merging. The validator was simplified and moved to the application layer instead of being added to craft-parts.
qwen3.6-35b-a3b-mtp-q6 Closed without merging. The proposed validator was superseded as functionality was simplified and relocated to the application layer instead of craft-parts.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#70 parts: use part validator from craft-parts rockcraft merged Merged after approval and rebase. Replaced the local part validator with the shared implementation from craft-parts to validate part data against part and plugin models, removing the deprecated local code.
74%
#3909 parts: use part validator from craft-parts snapcraft merged Merged into main. Replaced the local part validator with the shared craft-parts implementation, removing 119 lines across six files. Approved by reviewers, passed CI, and increased test coverage by 0.80%.
72%
#277 parts: add part validator craft-parts merged Merged adds a centralized part validation helper that validates data against part and plugin models, replacing per-tool logic. Approved, passed CI, and resolves issue #176.
71%