From 124c5d02e5ba25dcd60bfdaafbebfc980a3bde38 Mon Sep 17 00:00:00 2001 From: Anders Kaseorg Date: Wed, 23 Aug 2023 14:05:54 -0700 Subject: [PATCH] ci: Restore commented clean_unused_caches.py invocation. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The comment logic doesn’t make sense. Every build gets to write to the caches; some builds do in fact add new items, and without clean_unused_caches.py there’s no way for them to remove items. Signed-off-by: Anders Kaseorg --- .github/workflows/zulip-ci.yml | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/.github/workflows/zulip-ci.yml b/.github/workflows/zulip-ci.yml index 86c58cab01..0b69c5d510 100644 --- a/.github/workflows/zulip-ci.yml +++ b/.github/workflows/zulip-ci.yml @@ -100,10 +100,7 @@ jobs: run: | # This is the main setup job for the test suite ./tools/ci/setup-backend --skip-dev-db-build - - # Cleaning caches is mostly unnecessary in GitHub Actions, because - # most builds don't get to write to the cache. - # scripts/lib/clean_unused_caches.py --verbose --threshold 0 + scripts/lib/clean_unused_caches.py --verbose --threshold=0 - name: Run tools test run: |