Move Git abstraction outside of remotebuild
Metadata
Current evaluation
Moved GitRepo from remotebuild into craft_application.git in canonical/craft-application#388, merged as ea697712; fixes #385.
Resolution: closed resolved
Reason: PR canonical/craft-application#388 was merged as commit ea697712, moving GitRepo into craft_application/git/_git_repo.py and explicitly fixing #385 (CRAFT-3123).
Evaluated by: mr-cal-bot
No scores available.
Issue body
### What needs to get done
Create a separate module for handling git repositories.
Move the code related to git repository handling from `remotebuild` to the new `git` module.
### Why it needs to get done
Currently Git repository handling is a part of `remotebuild` module.
There is a need for a common place for abstraction around git repository handling, that may be reused in different places.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Moved GitRepo from remotebuild into craft_application.git in canonical/craft-application#388, merged as ea697712; fixes #385. | |
| stealth/space-bunny-alpha | — | closed stale | Closed without a documented code change; the only update was an automated CRAFT-3123 Jira-ticket notification. | |
| stealth/space-bunny-alpha | — | closed superseded | Closed after transfer to internal Jira ticket CRAFT-3123; no public PR, commit, or released module documents completion of the Git abstraction extraction. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed and superseded by internal Jira ticket CRAFT-3123. No public implementation or merge details provided. The request to extract git handling into a separate module remains unaddressed. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed and superseded by internal Jira ticket CRAFT-3123. No pull requests or code changes were created to extract the git handling module from remotebuild. The request remains unimplemented. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/craft-application#388
(confidence 100%)
Merged PR moved GitRepo out of remotebuild into the separate craft_application.git module.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #388 chore: move GitRepo to separate module | craft-application | merged | PR #388 by dariuszd21 was merged as commit ea697712, moving GitRepo out of the remotebuild code into its own craft_application/git/_git_repo.py module. Fixes #385 (CRAFT-3123). | |
| #576 feat: extend craft_application.git module | craft-application | merged | PR #576 merged, extending GitRepo with remote configuration, commit checks, last-commit lookup, fetch, and craft.git integration; Ruff follow-up #580 was also merged. | |
| #396 feat: add clone and remotes to GitRepo | craft-application | merged | Merged #396 in commit 90423a82, adding GitRepo cloning plus remote creation, renaming, and existence checks (CRAFT-3087). | |
| #5223 remote-build: refactor onto craft-application's command | snapcraft | closed | Remote-build was refactored onto craft-application's RemoteBuild in merged PR #5239 (commit f31d99498), bringing the requested upstream improvements. | |
| #4368 feat(remote-build): add GitRepo class | snapcraft | merged | Merged PR #4368 as dc58181e8, adding a pygit2-based GitRepo wrapper for remote builds and resolving #4320 (CRAFT-1975). |