From 9be23abcba4b734a4c05be9fc08d97bec5653dbb Mon Sep 17 00:00:00 2001 From: Kent Huang Date: Wed, 8 May 2024 10:41:44 +0800 Subject: [PATCH 1/3] [Feature] Update Recce CI workflow script Signed-off-by: Kent Huang --- .github/workflows/recce_ci.yml | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/.github/workflows/recce_ci.yml b/.github/workflows/recce_ci.yml index e825c57f6..cd91da11d 100644 --- a/.github/workflows/recce_ci.yml +++ b/.github/workflows/recce_ci.yml @@ -22,7 +22,7 @@ jobs: pip install -r requirements.txt - name: Install Recce run: | - pip install recce-nightly==0.15.0.20240426a389 + pip install recce-nightly - name: Add packages.yml file run: | echo '${{ vars.PACKAGES_YAML }}' > packages.yml @@ -73,6 +73,18 @@ jobs: recce summary recce_state.json > recce_summary.md cat recce_summary.md >> $GITHUB_STEP_SUMMARY echo '${{ env.NEXT_STEP_MESSAGE }}' >> recce_summary.md + + # Handle the case when the recce summary is too long to be displayed in the GitHub PR comment + if [[ `wc -c recce_summary.md | awk '{print $1}'` -ge '65535' ]]; then + cat << EOF + # Recce Summary + The recce summary is too long to be displayed in the GitHub PR comment. + Please check the summary detail in the [Job Summary](${{github.server_url}}/${{github.repository}}/actions/runs/${{github.run_id}}) page. + + ${{ env.NEXT_STEP_MESSAGE }} + EOF > recce_summary.md + fi + env: ARTIFACT_URL: ${{ steps.recce-artifact-uploader.outputs.artifact-url }} NEXT_STEP_MESSAGE: | From 548ddceb3c2fc83462c4ef71752d1435e3bc7d3b Mon Sep 17 00:00:00 2001 From: Kent Huang Date: Wed, 8 May 2024 11:15:42 +0800 Subject: [PATCH 2/3] [Fix] Change to use latest nightly build Signed-off-by: Kent Huang --- .github/workflows/recce_ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/recce_ci.yml b/.github/workflows/recce_ci.yml index cd91da11d..c1c79b68c 100644 --- a/.github/workflows/recce_ci.yml +++ b/.github/workflows/recce_ci.yml @@ -22,7 +22,7 @@ jobs: pip install -r requirements.txt - name: Install Recce run: | - pip install recce-nightly + pip install recce-nightly==0.17.0.20240508a0 - name: Add packages.yml file run: | echo '${{ vars.PACKAGES_YAML }}' > packages.yml From ba7047c58ed58cf3ad9966c4a659807600985d25 Mon Sep 17 00:00:00 2001 From: Kent Huang Date: Wed, 8 May 2024 11:21:10 +0800 Subject: [PATCH 3/3] [Fix] message wording Signed-off-by: Kent Huang --- .github/workflows/recce_ci.yml | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/.github/workflows/recce_ci.yml b/.github/workflows/recce_ci.yml index c1c79b68c..f9270e6fe 100644 --- a/.github/workflows/recce_ci.yml +++ b/.github/workflows/recce_ci.yml @@ -76,13 +76,10 @@ jobs: # Handle the case when the recce summary is too long to be displayed in the GitHub PR comment if [[ `wc -c recce_summary.md | awk '{print $1}'` -ge '65535' ]]; then - cat << EOF - # Recce Summary - The recce summary is too long to be displayed in the GitHub PR comment. - Please check the summary detail in the [Job Summary](${{github.server_url}}/${{github.repository}}/actions/runs/${{github.run_id}}) page. - - ${{ env.NEXT_STEP_MESSAGE }} - EOF > recce_summary.md + echo '# Recce Summary + The recce summary is too long to be displayed in the GitHub PR comment. + Please check the summary detail in the [Job Summary](${{github.server_url}}/${{github.repository}}/actions/runs/${{github.run_id}}) page. + ${{ env.NEXT_STEP_MESSAGE }}' > recce_summary.md fi env: