states: add pull state class
Metadata
Current evaluation
Merged PR #13 via df8666c2, adding PullState to retain pull-step context and track relevant properties and project options; it partially replaced PR #12.
Resolution: closed resolved
Reason: PR #13 was merged through commit df8666c2, with implementation commit c12b1827 adding PullState. Its work partially replaced the closed, unmerged PR #12.
Evaluated by: mr-cal-bot
No scores available.
Issue body
The PullState class holds context information collected when the
pull step runs for a part.
This PR partially replaces PR #12.
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 #13 via df8666c2, adding PullState to retain pull-step context and track relevant properties and project options; it partially replaced PR #12. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #13 via df8666c2, adding PullState to retain context collected during a part’s pull step; it partially replaced PR #12. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged the PullState class addition to store pull step context, partially replacing PR #12. Approved by two reviewers, the change modifies 7 files with 265 additions and 10 deletions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged the addition of a PullState class to store context information during a part's pull step. This change partially replaces PR #12. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/craft-parts#12
(confidence 99%)
PR #13 implemented the pull-state portion of the broader state-class work proposed in #12.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #12 states: add concrete step states | craft-parts | closed | Closed unmerged and split into #13–#16, which separately added PullState, BuildState, StageState, and PrimeState; all four replacement PRs were merged. | |
| #14 states: add build state class | craft-parts | merged | Merged PR #14 added BuildState for context collected during part build steps, partially replacing unmerged PR #12 in merge commit 373d3a84. | |
| #16 states: add prime state class and state loader | craft-parts | merged | Merged PR #16 added PrimeState and persistent step-state loading; commit aa062828 landed via merge 524909c6, replacing unmerged PR #12. | |
| #10 states: add step state base class | craft-parts | merged | Merged PR #10 via commit 35d2dde5, adding a base class for step state that preserves execution context, detects relevant changes, and enables unchanged runs to be skipped. |