feat: extend craft_application.git module
Metadata
Current evaluation
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.
Resolution: closed resolved
Reason: PR canonical/craft-application#576 was approved and merged, completing the GitRepo enhancements. Its cited Ruff cleanup was merged separately in #580 at commit 97347705.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [ ] Have you followed the guidelines for contributing?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox`?
-----
Add ability to manipulate repository and get additional data through `GitRepo` class:
- `set_remote_url` / `set_remote_push_url` alongside `get` counterparts (also `set_no_push` for disabling push for specific remotes)
- `is_commit()`, `is_short_commit` checkers
- use `craft.git` for subprocess git operation if available
- `get_last_commit()` to retrieve last commit on the current `HEAD`
- `fetch()` API to retrieve remote references
Linter issues fixed in #580
(CRAFT-3708)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | 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. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #576 expanded GitRepo with remote mutation, commit validation, last-commit lookup, fetch support, and craft.git-backed subprocesses; lint fixes from #580 were included. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #576 extended GitRepo with remote-setting, commit-checking, last-commit, fetch, and craft.git integration APIs; lint fixes were included via #580 (CRAFT-3708). | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval from three reviewers and passing CI checks. The update extends the craft_application.git module by adding GitRepo methods for remote URL management, commit verification, fetch operations, and subprocess integration. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/craft-application#580
(confidence 99%)
Merged Ruff fixes for lint issues introduced by #576.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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). | |
| #385 Move Git abstraction outside of remotebuild | craft-application | closed | Moved GitRepo from remotebuild into craft_application.git in canonical/craft-application#388, merged as ea697712; fixes #385. | |
| #925 feat: use "craft.git" for git where possible | craft-parts | merged | Merged in canonical/craft-parts#925 via commit 11be8c7a, preferring craft.git when available and falling back to git; fixes #900. |