parts: better error message when defining parts
Metadata
Current evaluation
Merged improved error messaging for part definitions. Approved by one reviewer, passed CI, and changed two files (+7/-3 lines). Addresses LP #1604827.
Suggested action: —
No scores available.
Issue body
LP: #1604827
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged improved error messaging for part definitions. Approved by one reviewer, passed CI, and changed two files (+7/-3 lines). Addresses LP #1604827. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Resolves LP #1604827 by improving error messages during part definition. Maintains 96.33% overall coverage with 100% diff coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Enhances error messaging for part definitions, resolving LP #1604827. Maintains 96.33% code coverage with no diff impact. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #569 fix(parts): validate part dependency name | craft-parts | merged | Merged code that validates part dependency names, raising an error for non-existent references. Approved by two reviewers, passed CI, and resolves issue #539. | |
| #922 fix: expose part deps in PartInfo | craft-parts | merged | Merged pull request exposing part dependencies in PartInfo. Approved by two reviewers and passed CI. Modified 11 files (+51/-16), initially addressing linting issues before implementing the core fix. | |
| #2577 docs: re-enable 'Part properties' | charmcraft | merged | Merged documentation update re-enabling 'Part properties' in the Parts index. Approved by two reviewers and passed CI. Resolves #2378 and partly addresses #2554. Minor type-check warnings were noted as non-blocking. | |
| #1379 docs: fix 'Part properties' typos | craft-parts | merged | Merged documentation update correcting typos and inconsistencies in the 'Part properties' reference. Approved by two reviewers with all CI checks passing. Changes modified three files. | |
| #558 fix(parts): validate part dependency name | craft-parts | merged | Merged fix for #539 that validates part dependency names. The change raises an error when a part references a non-existent dependency, approved by two reviewers and passing CI checks. | |
| #125 main: add details to the parts specification error message (CRAFT-512) | craft-parts | merged | Merged enhancement to craft-parts CLI that adds specific validation failure details to invalid parts specification errors. Approved by two reviewers and passed CI checks. | |
| #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. | |
| #541 Improve the error message when a part binary is not found. | snapcraft | merged | Merged to improve the error message when a part binary is not found, addressing LP: #1582367. The update modifies two files with minor adjustments to clarify missing binary errors. | |
| #527 Add more info about reusable parts. | snapcraft | merged | Merged documentation update adding links to the parts wiki and a section on reusing parts. Approved by contributors and merged after verification, resolving LP #1582499. |