From 77aa26bcdb8274955a0b7647a512eeb3f4e69def Mon Sep 17 00:00:00 2001 From: David Baker Date: Wed, 5 Nov 2025 13:55:19 +0000 Subject: [PATCH] Remove duplicate test step (#31166) As the comment indicated, this apparently was necessary to get stuff built the first time, but does not appear to be necessary anymore and looks to be causing occasional flakiness, so let's remove it. --- .github/workflows/shared-component-visual-tests.yaml | 6 ------ 1 file changed, 6 deletions(-) diff --git a/.github/workflows/shared-component-visual-tests.yaml b/.github/workflows/shared-component-visual-tests.yaml index 0377c96c7f..aa9374f934 100644 --- a/.github/workflows/shared-component-visual-tests.yaml +++ b/.github/workflows/shared-component-visual-tests.yaml @@ -55,12 +55,6 @@ jobs: if: steps.playwright-cache.outputs.cache-hit != 'true' run: "yarn playwright install --with-deps --only-shell" - - name: Build storybook dependencies - # When the first test is ran, it will fail because the dependencies are not yet built. - # This step is to ensure that the dependencies are built before running the tests. - run: "yarn --cwd packages/shared-components test:storybook:ci" - continue-on-error: true - - name: Run Visual tests run: "yarn --cwd packages/shared-components test:storybook:ci"