Skip to content

fix: exclude tmp from yarn-project clean-lite - #24342

Merged
PhilWindle merged 1 commit into
merge-train/spartan-v5from
spl/exclude-tmp-from-clean-lite
Jun 29, 2026
Merged

fix: exclude tmp from yarn-project clean-lite#24342
PhilWindle merged 1 commit into
merge-train/spartan-v5from
spl/exclude-tmp-from-clean-lite

Conversation

@spalladino

Copy link
Copy Markdown
Contributor

The clean-lite step runs on every yarn-project bootstrap build (directly, and via the root make -> yarn-project target). It deletes git-ignored files (build artifacts) via:

git ls-files --ignored --others --exclude-standard | grep -vE '(node_modules/|^\.yarn/)' | xargs rm -rf

It already spared node_modules/ and .yarn/, but not the top-level yarn-project/tmp/ scratch directory, so local scratch files there were wiped on every build. This adds ^tmp/ to the exclusion so tmp/ survives normal builds.

Note: the full git clean (./bootstrap.sh clean) still wipes everything, as before — this only affects the build-time clean-lite.

The clean-lite step run on every yarn-project bootstrap build deletes
git-ignored files (build artifacts). It already spared node_modules and
.yarn; also spare the top-level tmp/ scratch directory so local scratch
files survive a build.
@PhilWindle
PhilWindle merged commit c941cf3 into merge-train/spartan-v5 Jun 29, 2026
12 checks passed
@PhilWindle
PhilWindle deleted the spl/exclude-tmp-from-clean-lite branch June 29, 2026 07:21
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.

2 participants