brooktrails/pr-reviewer
Watch
1
0
Fork
You've already forked pr-reviewer
 0
Code Issues 2 Pull requests Projects Releases Packages Wiki Activity Actions

fix(ci): drop redundant apt/git install step #82

Merged
rcsheets merged 1 commit from fix/ci-drop-redundant-apt-step into main 2026-07-24 06:00:11 +00:00
Conversation 1 Commits 1 Files changed 2 +8 -22
rcsheets commented 2026-07-24 05:57:15 +00:00
Owner
Copy link

golang:1.26 is built on buildpack-deps:*-scm and already ships git and
ca-certificates (verified: git 2.47.3), which is all the checkout step needs.
The "Install git" step added nothing but a hard dependency on the in-cluster
apt-cacher-ng proxy — so when that proxy started 404ing the Debian trixie
index files, CI failed at apt-get update before doing any work. Remove the
step (and the now-unused DEBIAN_FRONTEND env) from ci.yaml and fixtures.yaml;
leave a note so it isn't re-added. build-push.yaml is untouched — it genuinely
installs buildah/podman/jq/curl.

Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com

golang:1.26 is built on buildpack-deps:*-scm and already ships git and ca-certificates (verified: git 2.47.3), which is all the checkout step needs. The "Install git" step added nothing but a hard dependency on the in-cluster apt-cacher-ng proxy — so when that proxy started 404ing the Debian trixie index files, CI failed at apt-get update before doing any work. Remove the step (and the now-unused DEBIAN_FRONTEND env) from ci.yaml and fixtures.yaml; leave a note so it isn't re-added. build-push.yaml is untouched — it genuinely installs buildah/podman/jq/curl. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
fix(ci): drop redundant apt/git install step
All checks were successful
ci / check (pull_request) Successful in 42s
Details
9ee5d08485 
golang:1.26 is built on buildpack-deps:*-scm and already ships git and
ca-certificates (verified: git 2.47.3), which is all the checkout step needs.
The "Install git" step added nothing but a hard dependency on the in-cluster
apt-cacher-ng proxy — so when that proxy started 404ing the Debian trixie
index files, CI failed at apt-get update before doing any work. Remove the
step (and the now-unused DEBIAN_FRONTEND env) from ci.yaml and fixtures.yaml;
leave a note so it isn't re-added. build-push.yaml is untouched — it genuinely
installs buildah/podman/jq/curl.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
pr-reviewer-bot commented 2026-07-24 05:57:16 +00:00
Collaborator
Copy link

Automated review by pr-reviewer v0.40.0 | Safety Check | Mistral Small | tracking id r-62febc-71bc8f
This is an AI-generated review and may contain mistakes.

Status: ✅ Completed

✅ Verdict: LGTM — No issues found

No obvious bugs, security issues, or accidentally committed files detected.

*Automated review by [pr-reviewer](https://git.brooktrails.org/brooktrails/pr-reviewer) v0.40.0 | Safety Check | Mistral Small | tracking id `r-62febc-71bc8f`* *This is an AI-generated review and may contain mistakes.* **Status:** ✅ Completed --- **✅ Verdict: LGTM** — No issues found No obvious bugs, security issues, or accidentally committed files detected.
rcsheets deleted branch fix/ci-drop-redundant-apt-step 2026-07-24 06:00:11 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
brooktrails/pr-reviewer!82
No description provided.