chore: use pathlib to interact with craft-parts
Metadata
Current evaluation
Adapts rockcraft's craft-parts calls to pathlib after upstream API changes; +10/-12 across 4 files, CI passing, awaiting review for 249 days.
Suggested action: needs review
Reason: CI is passing (labeler only, no failing checks) and the diff is small (+10/-12 across 4 files), but the PR has sat unreviewed for 249 days with 0 comments. It adapts to the merged upstream craft-parts#1424 pathlib changes and should get maintainer review.
Evaluated by: mr-cal-bot
Impact:
40
Quick Win:
22
Complexity:
20
Confidence:
65
Actionability:
70
Issue body
Adjust calls to functions of craft-parts to use pathlib after [changes in craft-parts API](https://github.com/canonical/craft-parts/pull/1424).
---
- [X] I've followed the [contribution guidelines](https://github.com/canonical/rockcraft/blob/main/CONTRIBUTING.md).
- [X] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/).
- [X] I've successfully run `make lint && make test`.
- [ ] I've added or updated any relevant documentation.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| ~deepseek/deepseek-pro-latest |
Impact:
40
Quick Win:
22
Complexity:
20
Confidence:
65
Actionability:
70
|
needs review | Adapts rockcraft's craft-parts calls to pathlib after upstream API changes; +10/-12 across 4 files, CI passing, awaiting review for 249 days. | |
| qwen/qwen3.6-35b-a3b |
Impact:
10
Quick Win:
3
Complexity:
10
Confidence:
60
Actionability:
30
|
keep open | Small chore PR adapting rockcraft to craft-parts pathlib API changes (craft-parts#1424). 234 days old, no review activity, CI passing. | |
| qwen/qwen3.8-27b |
Impact:
20
Quick Win:
17
Staleness:
85
Complexity:
15
Confidence:
80
|
close stale | Adapts craft-parts API calls to use pathlib after upstream craft-parts#1424. 217 days old, zero comments/reviews. Same change already landed in main via commit 1406be57 'refactor: update for Path change in craft-parts'. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
80
Complexity:
10
Confidence:
75
|
needs review | Adapts craft-parts API calls to use pathlib following upstream changes. PR passes CI, has no review comments, and has been inactive for 176 days awaiting maintainer triage. | |
| qwen3.6-35b-a3b-mtp-q6 |
Staleness:
85
Complexity:
15
Confidence:
80
|
needs review | Adapts craft-parts API calls to use pathlib following upstream changes. Currently inactive with no maintainer feedback or CI status. | |
| qwen3.6-35b-a3b-mtp-q6 |
Staleness:
90
Complexity:
20
Confidence:
85
|
close stale | Adapts craft-parts API calls to use pathlib following upstream changes. Inactive for 159 days with no maintainer feedback or CI status. Likely superseded by upstream integration or project progression. |
Update history
No update history recorded yet.
Related work
-
Caused By:
canonical/craft-parts#1424
(confidence 95%)
Upstream craft-parts API change to pathlib that this PR adapts rockcraft to.
-
Related To:
canonical/snapcraft#5981
(confidence 90%)
Analogous snapcraft PR adapting to the same craft-parts pathlib changes; has unresolved review comments and failing CI.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5981 chore: use pathlib to interact with craft-parts | snapcraft | open | Pathlib migration for craft-parts is now redundant on main; the PR is inactive after 249 days with four unresolved comments and multiple failing checks. | |
| #272 chore: change mount calls to use pathlib | imagecraft | closed | Pathlib migration for craft-parts mount calls was closed unmerged after 196 days with no review, CI, comments, or resolving commit. | |
| #596 chore: point to temporary build dependencies | rockcraft | merged | Merged PR #596 in d09f1816, pointing craft-parts and craft-application dependencies to their temporary rockcraft branches. PR #624 later restored proper released versions. | |
| #1599 chore(deps): update dependency craft-parts to v1.27.0 | charmcraft | merged | Renovate PR updating charmcraft's craft-parts pin from ==1.26.2 to ==1.27.0 (2 files, +2/-2). Approved by 2 reviewers with no unresolved comments and merged to main via commit c6e5d545. |