Skip to content

Commit 48e34fb

Browse files
[CONFIGURATION] Adjusted CI/CD configs slightly
* These changes are part of the review of PR #528 Changes in file .github/actions/check-control/action.yml: * fixed a duplicate label Changes in file .github/workflows/Tests.yml: * improved stability slightly * improved GHA reporting slightly
1 parent 55033f7 commit 48e34fb

2 files changed

Lines changed: 32 additions & 3 deletions

File tree

.github/actions/check-control/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -321,7 +321,7 @@ runs:
321321
summary_input="${RAW_CHECK_SUMMARY_INPUT}" || summary_input="" ;
322322
sanitized_input_summary_field=$(printf "%s%s" 'output[summary]=' "$summary_input" ;)
323323
printf "%s\n" "::endgroup::"
324-
printf "%s\n" "::group::validate-summary"
324+
printf "%s\n" "::group::validate-description"
325325
text_input="${RAW_CHECK_TEXT_INPUT}" || text_input="" ;
326326
sanitized_input_text_field=$(printf "%s%s" 'output[text]=' "$text_input" ;)
327327
printf "%s\n" "::endgroup::"

.github/workflows/Tests.yml

Lines changed: 31 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -476,7 +476,9 @@ jobs:
476476
coveralls-token: ${{ github.server_url == 'https://github.com' && secrets.COVERALLS_REPO_TOKEN || '' }}
477477
- name: Generate Doctest Coverage for py${{ matrix.python-version }} on ${{ matrix.os }}
478478
id: doctests-main
479-
run: make -f Makefile test-mat-doctests ;
479+
shell: bash
480+
run: |
481+
make -f Makefile test-mat-doctests || exit 1
480482
- name: Upload Python ${{ matrix.python-version }} doctest coverage to Codecov
481483
id: doctests-codecov-upload
482484
uses: codecov/codecov-action@671740ac38dd9b0130fbe1cec585b89eea48d3de # v5.5.2
@@ -803,6 +805,31 @@ jobs:
803805
id: post-style-test
804806
run: make -j1 -f Makefile clean || true ;
805807
if: ${{ success() }}
808+
- name: "Evaluate Style-Tests"
809+
id: styletests_outcome
810+
if: ${{ !cancelled() }}
811+
shell: bash
812+
run: |
813+
if [[ "${{ steps.style.outcome }}" == "success" ]] ; then
814+
THE_RESULT="success"
815+
else
816+
THE_RESULT="failure"
817+
fi
818+
if [[ "${THE_RESULT}" == "success" ]] ; then
819+
exit 0
820+
else
821+
exit 1
822+
fi
823+
- name: "Summarize Syle-Tests"
824+
id: style_report
825+
if: ${{ always() }}
826+
shell: bash
827+
run: |
828+
if [[ "${{ steps.styletests_outcome.outcome }}" == "success" ]] ; then
829+
printf "%s\n" " ${PASS_ICON} Style Checks succeeded with python version \`${PYTHON_DEFAULT}\` for \`${VCS_COMMIT_ID}\`" >> "$GITHUB_STEP_SUMMARY"
830+
else
831+
printf "%s\n" " ${FAIL_ICON} ~Style Checks succeeded with python version \`${PYTHON_DEFAULT}\`~" >> "$GITHUB_STEP_SUMMARY"
832+
fi
806833
807834
808835
INTEGRATION:
@@ -892,7 +919,9 @@ jobs:
892919
shell: bash
893920
- name: Generate Doctest Coverage for py${{ matrix.python-version }} on ${{ matrix.os }}
894921
id: integration-doctests
895-
run: make -f Makefile test-mat-doctests ;
922+
shell: bash
923+
run: |
924+
make -f Makefile test-mat-doctests ;
896925
- name: Generate Functional Test Coverage for py${{ matrix.python-version }} on ${{ matrix.os }}
897926
id: integration-main
898927
run: |

0 commit comments

Comments
 (0)