Skip to content

revert(covgate): remove build-cache pre-warm (#34–#36)#40

Merged
ben-miru merged 3 commits into
mainfrom
revert/covgate-prewarm-build-cache
Jul 3, 2026
Merged

revert(covgate): remove build-cache pre-warm (#34–#36)#40
ben-miru merged 3 commits into
mainfrom
revert/covgate-prewarm-build-cache

Conversation

@ben-miru

@ben-miru ben-miru commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

What

Reverts the covgate build-cache pre-warm feature in its entirety. covgate no longer pre-warms the Go build cache before its parallel coverage runs.

Reverts, in order (newest first):

Why

We no longer want covgate to pre-warm the build cache. The pre-warm compiled an extra artifact set that inflated the CI gobuild-test cache and pushed the backend repo back over the 10 GB Actions cache quota (the very regression #36 tried to walk back). Rather than keep iterating on the pre-warm, we're removing it.

How

Three git revert commits, preserving history/attribution. The reverted internal/services/covgate/ and internal/services/gocover/ files are byte-identical to their pre-feature state, and the feature's design doc (plans/completed/20260624-covgate-prewarm-build-cache.md) is removed.

Validation

🤖 Generated with Claude Code


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

@ben-miru
ben-miru merged commit dd4aaf0 into main Jul 3, 2026
3 checks passed
@ben-miru
ben-miru deleted the revert/covgate-prewarm-build-cache branch July 3, 2026 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant