chore(repo): untrack target-stats/, 6,624 committed build artifacts - #590
Merged
Conversation
`target-stats/` is a CARGO_TARGET_DIR that was committed by accident in #431. It is not in .gitignore, so all 6,624 of its build artifacts are tracked — 64% of the repository's 10,315 tracked files, roughly 1 GB. Because they are tracked, git writes every one of them on every clone and on every `git worktree add`. That is not a cosmetic problem: - Two agent worktree creations failed outright today with "No space left on device" while git was writing target-stats/release-fast/deps/*.rlib. - The same full disk took out the OrbStack moon-dev VM, which made both VM legs of scripts/ci-local.sh report FAIL with zero failing tests — an infrastructure failure wearing the costume of a code failure. - Every contributor pays ~1 GB of stale artifacts on clone, and the artifacts are from a 2026-08-04 toolchain, so they are useless to anyone. This removes them from the index (the working-tree copies are build output and are regenerated on demand) and adds a `target-*/` rule so no future CARGO_TARGET_DIR can be committed the same way. The existing narrow rules are kept for clarity. Verified before committing: - 6,624 staged deletions, of which 0 are outside target-stats/ - no .rs source file is deleted - nothing in Cargo.toml, .cargo/config.toml, scripts/, or .github/ references target-stats author: Tin Dang
|
ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing |
|
Caution CodeRabbit couldn't post its review summary. Error details |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
target-stats/is aCARGO_TARGET_DIRcommitted by accident in #431. It is not gitignored, so all 6,624 of its build artifacts are tracked — 64% of the repo's 10,315 tracked files, roughly 1 GB.Because they are tracked, git writes every one on every clone and every
git worktree add. That is not cosmetic:No space left on devicewhile git was writingtarget-stats/release-fast/deps/*.rlib.moon-devVM, which made both VM legs ofscripts/ci-local.shreport FAIL with zero failing tests — an infrastructure failure wearing the costume of a code failure. That is the most expensive part: it makes the local merge gate untrustworthy.Change
git rm -r --cached target-stats— removes from the index; working-tree copies are build output, regenerated on demand.target-*/rule so no futureCARGO_TARGET_DIRcan be committed the same way. Existing narrow rules kept for clarity.Verified before committing
target-stats/.rssource files deletedCargo.toml/.cargo/config.toml/scripts//.github/No source, config, or CI file is touched.
skip-changelogapplies — this changes no product behaviour.