From dc873bd07f7ad9b24763ef75e5ce849210dc2c8d Mon Sep 17 00:00:00 2001 From: Fran Mulero Date: Tue, 4 Oct 2022 11:49:26 +0200 Subject: [PATCH] Revert "[bitnami/containers] Fix result check in automatic PRs" (#8873) Revert "[bitnami/containers] Fix result check in automatic PRs (#8772)" This reverts commit 7e82c9db4678160ef9601a800a0d017a460b5648. --- .github/workflows/ci-pipeline.yml | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/.github/workflows/ci-pipeline.yml b/.github/workflows/ci-pipeline.yml index 3a79ab61d906..d5cae8a80dbb 100644 --- a/.github/workflows/ci-pipeline.yml +++ b/.github/workflows/ci-pipeline.yml @@ -114,12 +114,9 @@ jobs: needs: - get-containers - vib-verify - outputs: - result: ${{ steps.get-status.outputs.result }} if: ${{ always() }} steps: - - id: get-status - name: Check Status + - name: Check Status uses: actions/github-script@v6 with: script: | @@ -139,7 +136,6 @@ jobs: } else { core.notice(description) } - core.setOutput('result', state) try { await github.rest.repos.createCommitStatus({ context: `${context.workflow} / Verification Summary (${context.eventName})`, @@ -165,7 +161,7 @@ jobs: github.event.pull_request.user.login == 'bitnami-bot' steps: - name: Enable auto-merge feature - if: ${{ needs.verification-summary.outputs.result == 'success' }} + if: ${{ needs.verification-summary.result == 'success' }} uses: reitermarkus/automerge@v2.1.2 with: # Necessary to trigger CD workflow @@ -173,12 +169,12 @@ jobs: merge-method: squash pull-request: ${{ github.event.number }} - name: PR Approval - if: ${{ needs.verification-summary.outputs.result == 'success' }} + if: ${{ needs.verification-summary.result == 'success' }} uses: hmarr/auto-approve-action@v2.4.0 with: pull-request-number: ${{ github.event.number }} - name: Manual review required - if: ${{ needs.verification-summary.outputs.result != 'success' }} + if: ${{ needs.verification-summary.result != 'success' }} uses: peter-evans/create-or-update-comment@v2.0.0 with: issue-number: ${{ github.event.number }}