From 9ba510eb3fca9e7f5485a8eb90315f1ec28e1538 Mon Sep 17 00:00:00 2001 From: Paul Rastoin <45004772+prastoin@users.noreply.github.com> Date: Wed, 15 Jan 2025 11:59:17 +0100 Subject: [PATCH] [CI] Always save cache agnostically of the actions/restore cache-hit output (#9635) ## Motivations This is not because we've hit the `actions/cache/restore` cache that NX won't be re-building sub commands. Which means the grain to save or not the cache should be extracted from nx sub commands and not from the `actions/cache` outputs. We should investigate on the way to retrieve this granularity level From the moment we will be saving everything, which could result sometimes in duplicating cache instances. We should keep in mind that our cache occurrences have a 1 day retention date, is low cost and pretty fast to perform --- .github/workflows/ci-front.yaml | 4 ++-- .github/workflows/ci-server.yaml | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/ci-front.yaml b/.github/workflows/ci-front.yaml index da9fa86b1..234e41ff1 100644 --- a/.github/workflows/ci-front.yaml +++ b/.github/workflows/ci-front.yaml @@ -61,7 +61,7 @@ jobs: if: steps.changed-files.outputs.any_changed == 'true' run: npx nx storybook:build twenty-front - name: Save storybook build cache - if: steps.changed-files.outputs.any_changed == 'true' && steps.restore-storybook-build-cache.outputs.cache-hit != 'true' + if: steps.changed-files.outputs.any_changed == 'true' uses: ./.github/workflows/actions/save-cache with: key: ${{ steps.restore-storybook-build-cache.outputs.cache-primary-key }} @@ -252,7 +252,7 @@ jobs: tag: scope:frontend tasks: ${{ matrix.task }} - name: Save ${{ matrix.task }} cache - if: steps.changed-files.outputs.any_changed == 'true' && steps.restore-task-cache.outputs.cache-hit != 'true' + if: steps.changed-files.outputs.any_changed == 'true' uses: ./.github/workflows/actions/save-cache with: key: ${{ steps.restore-task-cache.outputs.cache-primary-key }} diff --git a/.github/workflows/ci-server.yaml b/.github/workflows/ci-server.yaml index 9e308adc0..58d3c9288 100644 --- a/.github/workflows/ci-server.yaml +++ b/.github/workflows/ci-server.yaml @@ -108,7 +108,7 @@ jobs: exit 1 fi - name: Save server setup - if: steps.changed-files.outputs.any_changed == 'true' && steps.restore-server-setup-cache.outputs.cache-hit != 'true' + if: steps.changed-files.outputs.any_changed == 'true' uses: ./.github/workflows/actions/save-cache with: key: ${{ steps.restore-server-setup-cache.outputs.cache-primary-key }}