From 52179ad70d8ff83f3745b7cddbf7e39335c8dd66 Mon Sep 17 00:00:00 2001 From: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com> Date: Wed, 27 Nov 2024 02:57:24 +0000 Subject: [PATCH] fix(ci): remove debug, smaller checkout --- .github/workflows/compare-layouts.yml | 3 --- .github/workflows/forge-ci.yml | 8 +++----- 2 files changed, 3 insertions(+), 8 deletions(-) diff --git a/.github/workflows/compare-layouts.yml b/.github/workflows/compare-layouts.yml index 5de8f660..08487eee 100644 --- a/.github/workflows/compare-layouts.yml +++ b/.github/workflows/compare-layouts.yml @@ -150,9 +150,6 @@ jobs: with: name: storage-layouts-${{ github.event.workflow_run.head_commit.id }} run_id: ${{ github.event.workflow_run.id }} - - name: Debug the restored layouts - if: ${{ github.event.workflow_run.conclusion == 'success' }} - run: ls -l - name: Extract the restored layouts if: ${{ github.event.workflow_run.conclusion == 'success' }} run: unzip storage-layouts.zip diff --git a/.github/workflows/forge-ci.yml b/.github/workflows/forge-ci.yml index f0c88b29..388d09d9 100644 --- a/.github/workflows/forge-ci.yml +++ b/.github/workflows/forge-ci.yml @@ -67,8 +67,6 @@ jobs: run: echo "${{ needs.build.outputs.installation-dir }}" >> $GITHUB_PATH - name: Checkout repository uses: actions/checkout@v4 - with: - submodules: recursive - name: Restore build artifacts uses: actions/cache/restore@v3 with: @@ -96,8 +94,6 @@ jobs: run: echo "${{ needs.build.outputs.installation-dir }}" >> $GITHUB_PATH - name: Checkout repository uses: actions/checkout@v4 - with: - submodules: recursive - name: Restore build artifacts uses: actions/cache/restore@v3 with: @@ -125,6 +121,9 @@ jobs: uses: actions/checkout@v4 with: ref: ${{ github.event.pull_request.base.ref || github.event.before }} + # We don't have a `lib` folder to restore for this step, so we + # recursively checkout the submodules. In other steps, we use the + # `lib` folder from the `build` job. submodules: recursive - name: Generate base branch layout file # Note that this `run` will do a `forge build` so we don't need to do it ourselves. @@ -139,7 +138,6 @@ jobs: uses: actions/checkout@v4 with: ref: ${{ github.event.pull_request.head.sha }} - submodules: recursive # Restoring these will help make `forge inspect` faster. - name: Restore build artifacts uses: actions/cache/restore@v3