deltas: Improved message returned when delta is too big
Metadata
Current evaluation
Merged an external contribution improving the oversized delta warning to display delta size before pushing. After fixing minor linting errors and resolving CLA verification issues, reviewers approved and merged the change.
Suggested action: —
No scores available.
Issue body
Improved the big delta warning/error message when the user is trying to push changes, making it display the size of the delta before pushing the full snap.
LP: [1681692](https://bugs.launchpad.net/snapcraft/+bug/1681692)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged an external contribution improving the oversized delta warning to display delta size before pushing. After fixing minor linting errors and resolving CLA verification issues, reviewers approved and merged the change. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Improves oversized delta warnings by showing delta size before pushing a full snap. Contributor resolved CLA requirements, fixed a PEP8 error, and adjusted GitHub email privacy to pass automated checks before merging. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Improved delta size warning message to display delta size before pushing. Merged after contributor license agreement signing, a missing blank line fix, and public email visibility for automated CLA checks. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2661 deltas: code cleanup | snapcraft | merged | Merged into master following reviewer approval and passing CI. Refactored the deltas module by implementing prominent error classes, replacing runtime checks with type hints, removing duplicated logic, and moving delta tests to a dedicated package. | |
| #1681692 Snap delta too large warning language can be improved | snapcraft (launchpad) | open | Request to simplify the 'delta generated was too large' warning to a single line with an explicit threshold. Open 9+ years with zero comments and no maintainer interaction; the exact warning text no longer appears in the current codebase. | |
| #934 deltas: migrate from xdelta to xdelta3. | snapcraft | merged | Merged migration from xdelta to xdelta3 for delta generation. Approved by one reviewer, the change updated 9 files (+49/-53 lines) and resolved launchpad issue #1645456. |