← Back to issue list

feat: improve error message for checksum mismatch

View original Github issue

Metadata

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

Current evaluation

Merged improvements to checksum mismatch error messages, adding actual and expected digests to details, a resolution entry, and the problematic part name. Approved by two reviewers with all CI checks passing. Resolves #355.

Suggested action:

No scores available.

Issue body

Add a better checksum error message, move actual and expected digest to details, provide a resolution entry, and show the name of the part causing the problem. This provides better UX in case of checksum mismatches and makes it clear where the error originates from. Fixes #355 - [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint && make test`? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? ---

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged improvements to checksum mismatch error messages, adding actual and expected digests to details, a resolution entry, and the problematic part name. Approved by two reviewers with all CI checks passing. Resolves #355.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 15
Confidence: 95
needs review Improves checksum mismatch error messages by adding digests, resolution steps, and part names. Approved by 2 reviewers, all CI checks passing, ready for maintainer merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 35
Confidence: 90
needs review Improves checksum mismatch error messages by including actual and expected digests, resolution steps, and the problematic part name. Currently approved by one reviewer, CI passing, and ready for final merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 15
Confidence: 90
needs review Improves checksum mismatch error messages by adding expected/actual digests, resolution steps, and part names. Currently approved by one reviewer, CI passing, awaiting final maintainer review.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 15
Confidence: 90
needs review Improves checksum mismatch error messages by adding digests, resolution steps, and part names. Approved by 1 reviewer, CI passing, but lint/test and changelog checkboxes remain unchecked.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 20
Confidence: 85
needs review Improves checksum mismatch error messages by adding digests, resolution steps, and part names. Currently awaiting initial maintainer review with CI checks passing or pending. Fixes #355.

Update history

Date Change
updated
updated
updated
updated
updated
created

Related issues

Issue Project State Summary Similarity
#60 feat: add checksum file to control tarball debcraft merged Merged to add a checksum file to the control tarball. Approved by two reviewers with all CI checks passing. Changes modified three files, adding 80 lines and removing 8.
74%
#26 sources: add support for source checksum craft-parts merged Merged: adds file checksum computation to validate the source-checksum part property. Approved by two reviewers and passed CI checks.
72%