From b05941cdb5e91b723c4d67461fa8778439bda3bf Mon Sep 17 00:00:00 2001 From: Denis Rouzaud Date: Thu, 11 Jul 2024 13:18:29 +0200 Subject: [PATCH] remove useless code from workflow (#58065) * remove useless code from workflow This was never used because of limitations of running on PRs. Other approaches have been used since. * comment set-output --- .ci/ctest2ci.py | 4 +-- .github/workflows/run-tests.yml | 53 --------------------------------- 2 files changed, 2 insertions(+), 55 deletions(-) diff --git a/.ci/ctest2ci.py b/.ci/ctest2ci.py index f0722f7216c..9a6ee626ef7 100755 --- a/.ci/ctest2ci.py +++ b/.ci/ctest2ci.py @@ -111,7 +111,7 @@ for line in p.stdout: if not in_failing_test and re.search('[0-9]+% tests passed, [0-9]+ tests failed out of', updated_line): tests_failing = re.match(r'.* ([0-9]+) tests failed', updated_line).group(1) - updated_line += '\n::set-output name=TESTS_FAILING::{}'.format(tests_failing) + # updated_line += '\n::set-output name=TESTS_FAILING::{}'.format(tests_failing) end_fold() if re.search('100% tests passed', updated_line): @@ -121,7 +121,7 @@ for line in p.stdout: start_fold('submit') elif re.search('Test results submitted to', updated_line): cdash_url = re.match(r'.*(http.*)$', updated_line).group(1) - updated_line += '\n::set-output name=CDASH_URL::{}'.format(cdash_url) + # updated_line += '\n::set-output name=CDASH_URL::{}'.format(cdash_url) end_fold() sys.stdout.write(updated_line) diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml index 7d2cc56a3ed..0c0ea4be58c 100644 --- a/.github/workflows/run-tests.yml +++ b/.github/workflows/run-tests.yml @@ -227,59 +227,6 @@ jobs: # set -e # switch back # docker stop qgis-testing-environment - -# tests-report-comment: -# name: Write tests report in a comment -# needs: build -# runs-on: ubuntu-latest -# if: always() && ( needs.build.result == 'failure' || needs.build.result == 'success' ) && github.event_name == 'pull_request' -# steps: -# - name: Find Comment -# uses: peter-evans/find-comment@v1 -# id: find-comment -# with: -# issue-number: ${{ github.event.pull_request.number }} -# comment-author: 'github-actions[bot]' -# body-includes: Tests report -# -# - name: Create comment -# if: ${{ steps.find-comment.outputs.comment-id == 0 }} -# uses: peter-evans/create-or-update-comment@v1 -# id: create-comment -# with: -# issue-number: ${{ github.event.pull_request.number }} -# body: | -# **Tests report** -# -# - name: Process -# id: process-vars -# env: -# COMMENT_FOUND: ${{ steps.find-comment.outputs.comment-id }} -# COMMENT_CREATED: ${{ steps.create-comment.outputs.comment-id }} -# COMMIT_SHA: ${{ github.event.pull_request.head.sha }} -# CDASH_URL: ${{ needs.build.outputs.cdash_url }} -# COMPILE_OUTCOME: ${{ needs.build.outputs.compile_outcome }} -# TESTS_FAILING: ${{ needs.build.outputs.tests_failing }} -# RUNNERS_OUTCOME: ${{ needs.build.outputs.runners_outcome }} -# run: | -# echo "::set-output name=COMMENT_ID::"$([[ "${COMMENT_FOUND}" -eq "0" ]] && echo ${COMMENT_CREATED} || echo ${COMMENT_FOUND}) -# if [[ ${COMPILE_OUTCOME} != "success" ]]; then -# echo "::set-output name=COMMENT_BODY::${COMMIT_SHA} :scream: compilation failed" -# elif [[ ${TESTS_FAILING} != "0" ]]; then -# echo "::set-output name=COMMENT_BODY::${COMMIT_SHA} :fire: ${TESTS_FAILING} unit-tests are failing ${CDASH_URL}" -# elif [[ ${RUNNERS_OUTCOME} != "success" ]]; then -# echo "::set-output name=COMMENT_BODY::${COMMIT_SHA} :broken_heart: QGIS runners test failed" -# else -# echo "::set-output name=COMMENT_BODY::${COMMIT_SHA} :sunglasses: unit-tests succeeded" -# fi -# -# - name: Update comment -# uses: peter-evans/create-or-update-comment@v1 -# with: -# comment-id: ${{ steps.process-vars.outputs.COMMENT_ID }} -# edit-mode: append -# body: ${{ steps.process-vars.outputs.COMMENT_BODY }} - run-tests: name: Run tests env: