Skip to content

Commit

Permalink
Issue checkstyle#15579: Issue of failure of make_report in new workfl…
Browse files Browse the repository at this point in the history
…ow does not make comment in PR
  • Loading branch information
piyush kumar sadangi authored and romani committed Aug 29, 2024
1 parent 3bb8d52 commit 6c1f0b4
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion .github/workflows/regression-report.yml
Original file line number Diff line number Diff line change
Expand Up @@ -561,7 +561,8 @@ jobs:
download_configs,
make_report ]
if: always()
&& (needs.make_report.outputs.message != ''
&& (needs.make_report.result == 'failure'
|| needs.make_report.outputs.message != ''
|| needs.download_configs.result == 'failure'
|| needs.handle_generated_config_bundle.result == 'failure')
steps:
Expand All @@ -572,6 +573,7 @@ jobs:
env:
MSG: ${{ needs.make_report.outputs.message }}
DOWNLOAD_RESULT: ${{ needs.download_configs.result }}
MAKE_REPORT_RESULT: ${{ needs.make_report.result }}
GENERATED_CONFIG_RESULT: ${{ needs.handle_generated_config_bundle.result }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: |
Expand All @@ -595,6 +597,11 @@ jobs:
echo "<br>Usage: 'GitHub, generate report by config from <file_path>'"
echo "<br>Link: $JOBS_LINK"
} > .ci-temp/message
elif [ "$MAKE_REPORT_RESULT" == "failure" ]; then
{
echo "Report generation failed. Please check the logs for more details."
echo "<br>Link: $JOBS_LINK"
} > .ci-temp/message
elif [ -z "$MSG" ]; then
API_LINK="https://api.github.com/repos/checkstyle/checkstyle/actions/runs/${{github.run_id}}/jobs"
curl --fail-with-body -X GET "${API_LINK}" \
Expand Down

0 comments on commit 6c1f0b4

Please sign in to comment.