Skip to content

Commit

Permalink
Merge pull request galaxyproject#5721 from mvdbeek/remove_comment_stuff
Browse files Browse the repository at this point in the history
Remove comment stuff, fix posting test reports when test(s) actually failed
  • Loading branch information
mvdbeek authored Jan 17, 2024
2 parents 2d21c3b + 6c59353 commit ada2a74
Showing 1 changed file with 1 addition and 22 deletions.
23 changes: 1 addition & 22 deletions .github/workflows/pr.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -367,6 +367,7 @@ jobs:
with:
name: 'All tool test results'
path: upload
- run: cat upload/tool_test_output.md >> $GITHUB_STEP_SUMMARY
- name: Check outputs
uses: galaxyproject/planemo-ci-action@v1
id: check
Expand All @@ -378,28 +379,6 @@ jobs:
if [[ "${{ needs.setup.outputs.chunk-count }}" != "$NFILES" ]]; then
exit 1
fi
- run: cat upload/tool_test_output.md >> $GITHUB_STEP_SUMMARY
# the following only works for PRs opened from the main fork
# which we should not do (but lets leave this here until we have a
# solution)
- name: Get PR object
if: failure()
uses: 8BitJonny/gh-get-current-pr@2.2.0
id: failed_pr_number
with:
sha: ${{ github.event.pull_request.head.sha }}
- name: Debug GetPR output
if: failure()
run: |
echo ${{ steps.failed_pr_number.outputs.number }}
- name: Post comment with test report
if: failure()
uses: peter-evans/create-or-update-comment@v3
with:
token: ${{ secrets.PAT }}
issue-number: ${{ steps.failed_pr_number.outputs.number }}
body-path: upload/tool_test_output.md

# deploy the tools to the toolsheds (first TTS for testing)
deploy:
Expand Down

0 comments on commit ada2a74

Please sign in to comment.