build: rename internal (non-release) Earthfile targets to earthbuild-*#647
build: rename internal (non-release) Earthfile targets to earthbuild-*#647kmannislands wants to merge 1 commit into
Conversation
Renames internal helper/test targets (definitions plus all BUILD/FROM references and the CONTRIBUTING.md mention): +earthly-script-no-stdout -> +earthbuild-script-no-stdout +earthly-integration-test-base -> +earthbuild-integration-test-base +build-earthly-test -> +build-earthbuild-test +test-earthly-mirror-was-setup -> +test-earthbuild-mirror-was-setup +test-works-without-earthly-server -> +test-works-without-earthbuild-server These are internal build/test targets only. Deliberately excludes the released-artifact / entry-point targets (+earthly, +earthly-docker, +earthly-linux-amd64, +earthly-all, etc.) which map to the published binary and image names. Changed Earthfiles parse via `earthly debug ast`; all definitions and their references remain consistent. Reduces "earthly" line count by 10. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
🎉 Are we earthbuild yet?Great progress! You've reduced "earthly" occurrences by 10 (0.18%) 📈 Overall Progress
📁 Changes by file type:
Keep up the great work migrating from Earthly to Earthbuild! 🚀 💡 Tips for finding more occurrencesRun locally to see detailed breakdown: ./.github/scripts/count-earthly.shNote that the goal is not to reach 0. |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request renames several targets and references from "earthly" to "earthbuild" across the codebase, including in CONTRIBUTING.md, Earthfile, and various test configurations. Feedback on the changes highlights that the renamed test-earthbuild-mirror-was-setup target is currently dead code because it is not referenced by any active test groups, and recommends adding it to the test suite so that it is executed.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| RUN cat output | perl -pe 'BEGIN {$status=1} END {exit $status} $status=0 if /\+run2 \| 2\n/;' | ||
|
|
||
| test-earthly-mirror-was-setup: | ||
| test-earthbuild-mirror-was-setup: |
There was a problem hiding this comment.
The test target test-earthbuild-mirror-was-setup (formerly test-earthly-mirror-was-setup) is defined here but is not referenced by any of the test groups (such as ga-no-qemu-group*) or any other target in the repository. This means this integration test is currently dead code and is never executed during the test suite run.\n\nPlease consider adding it to one of the active test groups (for example, ga-no-qemu-group12 or another appropriate group) so that it is run as part of the integration tests.
2 participants
What
Renames internal helper/test Earthfile targets (definitions + all
BUILD/FROMreferences + theCONTRIBUTING.mdmention):+earthly-script-no-stdout+earthbuild-script-no-stdout+earthly-integration-test-base+earthbuild-integration-test-base+build-earthly-test+build-earthbuild-test+test-earthly-mirror-was-setup+test-earthbuild-mirror-was-setup+test-works-without-earthly-server+test-works-without-earthbuild-serverScope / safety
Internal build/test targets only. Deliberately excludes the released-artifact / entry-point targets (
+earthly,+earthly-docker,+earthly-linux-amd64,+earthly-all, ...) which map to the published binary and image names and are referenced by release tooling.Verification
earthly debug ast; every definition and itsBUILD/FROMreferences remain consistent (verified no dangling refs).earthlyline count by 10.🤖 Generated with Claude Code