Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update entrypoint.sh #85

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Enrico-Infrinity
Copy link

@Enrico-Infrinity Enrico-Infrinity commented Apr 11, 2023

Deleted the line #196 ----> echo "conclusion=${conclusion}" >> $GITHUB_OUTPUT

Inserted new line #200 ----> echo "conclusion=${conclusion}" >> $GITHUB_OUTPUT

Inserted new line #204 ----> echo "conclusion=${conclusion}" >> $GITHUB_OUTPUT

With this simple update, the routine is able to write the $conclusion value on the $GITHUB_OUTPUT, and now is possible to setup the "propagate_failure: false", and look at the result of the remote workflow action. You can try the functionality with my version below :)

steps:
  - name: 'Run test'
     id: 'run-test'
     uses: Enrico-Infrinity/trigger-workflow-and-wait@v1.0
     with: owner: remoteOwner
     repo: remoteRepo
     github_token: ${{ secrets.CICD_REPO_TOKEN }}
     workflow_file_name: main.yml
     ref: main
     wait_interval: 10
     client_payload: '{}'
     propagate_failure: false
     trigger_workflow: true
     wait_workflow: true

  - name: 'Set result'
    run: |
      echo ${{ steps.run-test.outputs.conclusion }}

It's a simple workaround, but it's working well.

Enjoy!! :)

Deleted the line #196 ----> echo "conclusion=${conclusion}" >> $GITHUB_OUTPUT
Inserted new line #200 ----> echo "conclusion=${conclusion}" >> $GITHUB_OUTPUT
Inserted new line #204 ----> echo "conclusion=${conclusion}" >> $GITHUB_OUTPUT

With this simple update, the routine is able to write the $conclusion value on the $GITHUB_OUTPUT,
and now is possible to setup the "propagate_failure: false", and look at the result of the remote workflow action.
You can try the functionality with my version below :)
....
    steps:
      - name: 'Run test'
        id: 'run-test'
        uses: Enrico-Infrinity/trigger-workflow-and-wait@v1.0
        with:
          owner: remoteOwner
          repo: remoteRepo
          github_token: ${{ secrets.CICD_REPO_TOKEN }}
          workflow_file_name: main.yml
          ref: main
          wait_interval: 10
          client_payload: '{}'
          propagate_failure: false
          trigger_workflow: true
          wait_workflow: true

      - name: 'Set result'
        run: |
          echo ${{ steps.run-test.outputs.conclusion }}
....


It's a simple workaround, but it's working well.

Enjoy!! :)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants