Skip to content

Commit 337f2e4

Browse files
hjmjohnsonclaude
andcommitted
STYLE: Use pinned version for source dir removal and fix ccache comment
- Use ExternalDataVersion variable instead of broad InsightToolkit-* glob for source directory removal. - Fix misleading ccache --cleanup comment: it trims the cache to stay within CCACHE_MAXSIZE, not "flush to persistent cache dir". Addresses review comments from @greptile-apps on PR #6011. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 8511c76 commit 337f2e4

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/pixi.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -145,8 +145,8 @@ jobs:
145145
# Remove downloaded data tarballs (already extracted)
146146
rm -f InsightData-*.tar.gz
147147
# Remove extracted source tarball directory
148-
rm -rf InsightToolkit-*
149-
# Flush ccache to persistent cache dir and clear temp
148+
rm -rf InsightToolkit-${{ env.ExternalDataVersion }}
149+
# Trim ccache to stay within CCACHE_MAXSIZE and remove orphaned entries
150150
ccache --cleanup 2>/dev/null || true
151151
echo "****** df -h / -- post cleanup"
152152
df -h /

0 commit comments

Comments
 (0)