From b69d885ee62d2491628129178e52e86b82da55bd Mon Sep 17 00:00:00 2001 From: Keith Williams Date: Wed, 10 Jan 2024 11:35:27 -0300 Subject: [PATCH] fix: Bundle analysis commenting (#13116) * fix: Bundle analysis commenting * Remove if on build step for testing * Remove GitHub event number check for testing * Reverted changes for testing --- .github/workflows/nextjs-bundle-analysis.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/nextjs-bundle-analysis.yml b/.github/workflows/nextjs-bundle-analysis.yml index 4c8da86217..c92e6b2045 100644 --- a/.github/workflows/nextjs-bundle-analysis.yml +++ b/.github/workflows/nextjs-bundle-analysis.yml @@ -62,18 +62,18 @@ jobs: - name: Get comment body id: get-comment-body - if: success() + if: success() && github.event.number run: | cd apps/web body=$(cat .next/analyze/__bundle_analysis_comment.txt) body="${body//'%'/'%25'}" body="${body//$'\n'/'%0A'}" body="${body//$'\r'/'%0D'}" - echo "{name}={$body}" >> $GITHUB_OUTPUT + echo "{body}={$body}" >> $GITHUB_OUTPUT - name: Find Comment uses: peter-evans/find-comment@v2 - if: success() + if: success() && github.event.number id: fc with: issue-number: ${{ github.event.number }}