← Back to issue list

states: refactor step state class, add migration state (CRAFT-471)

View original Github issue

Metadata

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

Current evaluation

Merged PR #114 (commit 8703d588) refactored step-state handling and added a migration-state superclass for overlay-file migrations (CRAFT-471).

Resolution: closed resolved

Reason: The PR was merged into craft-parts, with repository metadata identifying commit 8703d588 as the resolving merge. Commit 139adbc3, titled "states: introduce superclass with migration state", implements the requested refactor and migration-state addition.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Refactor step state and add a migration state superclass to be used when migrating overlay files. 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
stealth/space-bunny-alpha — closed resolved Merged PR #114 (commit 8703d588) refactored step-state handling and added a migration-state superclass for overlay-file migrations (CRAFT-471).
stealth/space-bunny-alpha — closed resolved Merged in PR #114 (8703d588), refactoring step-state handling and adding a migration-state superclass for overlay-file migrations (CRAFT-471).
qwen/qwen3.6-35b-a3b — — Merged following approval by two reviewers. Refactors the step state class and introduces a migration state superclass for overlay file migrations. Modifies six files (+149/-38 lines).
qwen3.6-35b-a3b-mtp-q6 — — Merged refactor of the step state class and addition of a migration state superclass for overlay file migrations, addressing CRAFT-471.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#385 Migrate old state to new. snapcraft merged Merged PR #385 in c8ef2a54b (83c390289), adding migration from legacy state storage to the per-step tracking introduced by #381 for LP #1560158.
72%