meta: Snap to_dict() cleanup
Metadata
Current evaluation
Merged PR #2950 in commit 5934e0009, replacing __dict__-based Snap serialization with explicit properties and moving snap.yaml-specific output into to_snap_yaml_dict().
Resolution: closed resolved
Reason: PR #2950 was approved and merged as commit 5934e0009, completing the Snap.to_dict() cleanup.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- remove `__dict__` usage and detail each property
- remove snap.yaml-isms from to_dict, into newly added to_snap_yaml_dict()
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2950 in commit 5934e0009, replacing __dict__-based Snap serialization with explicit properties and moving snap.yaml-specific output into to_snap_yaml_dict(). | |
| stealth/space-bunny-alpha | — | closed resolved | PR #2950 merged a Snap.to_dict() cleanup: explicit properties replaced __dict__, while snap.yaml-specific serialization moved to the new to_snap_yaml_dict(). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #2950: replaced to_dict()'s __dict__ usage with explicit properties and moved snap.yaml-specific serialization to to_snap_yaml_dict(). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged cleanup in #2950 (commit 5934e0009): Snap.to_dict() now documents each property, removes __dict__ usage, and moves snap.yaml-specific behavior into to_snap_yaml_dict(). | |
| qwen/qwen3.6-35b-a3b | — | — | Merged refactoring that removed dict usage and snap yaml-specific logic from to_dict. Properties are now explicitly defined, with a new to_snap_yaml_dict method handling YAML serialization. Approved by one reviewer. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged cleanup of the Snap to_dict() method. Removed __dict__ usage and explicitly detailed each property. Relocated snap.yaml-specific logic to a new to_snap_yaml_dict() method. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2953 meta: remove remaining `__dict__` and key list usage in snap | snapcraft | merged | Merged PR #2953 in commit ef4d64e8d, removing remaining __dict__ and key-list usage from Snap metadata and fixing a Python 3.8 c901 label. | |
| #2821 meta: remove __dict__ usage in Snap | snapcraft | closed | Closed unmerged and superseded by smaller Snap refactor PRs after the author split the broad __dict__ cleanup to ease review. |