Skip to content

Commit

Permalink
Fix the url of Cirrus failure notifications [ci skip]
Browse files Browse the repository at this point in the history
The previous one was the url of this notification action, but we want to
see a link to Cirrus CI.

This code follows https://cirrus-ci.org/guide/notifications/.

Also, head_branch was null only for pull requests from a fork, so we can
use it for branch-based filtering, which is good.
  • Loading branch information
k0kubun committed Oct 26, 2022
1 parent c746f38 commit a8e2d73
Showing 1 changed file with 13 additions and 3 deletions.
16 changes: 13 additions & 3 deletions .github/workflows/cirrus-notify.yml
Expand Up @@ -10,23 +10,33 @@ jobs:
&& github.event.check_suite.conclusion != 'success'
&& github.event.check_suite.conclusion != 'cancelled'
&& github.event.check_suite.conclusion != 'skipped'
&& github.event.check_suite.head_branch == 'master'
runs-on: ubuntu-latest
steps:
- uses: octokit/request-action@v2.x
id: get_failed_check_run
with:
route: GET /repos/${{ github.repository }}/check-suites/${{ github.event.check_suite.id }}/check-runs?status=completed
mediaType: '{"previews": ["antiope"]}'
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- name: Dump GitHub context
env:
GITHUB_CONTEXT: ${{ toJson(github) }}
run: echo "$GITHUB_CONTEXT"
- name: Dump check_runs
env:
CHECK_RUNS: ${{ steps.get_failed_check_run.outputs.data }}
run: echo "$CHECK_RUNS"
- uses: ruby/action-slack@v3.0.0
with:
payload: |
{
"ci": "Cirrus CI",
"env": "Cirrus CI",
"url": "https://github.com/ruby/ruby/actions/runs/${{ github.run_id }}",
"url": "${{ fromJson(steps.get_failed_check_run.outputs.data).check_runs[0].html_url }}",
"commit": "${{ github.event.check_suite.head_commit.id }}",
"branch": "${{ github.event.check_suite.head_branch }}"
}
env:
SLACK_WEBHOOK_URL: ${{ secrets.SIMPLER_ALERTS_URL }} # ruby-lang slack: ruby/simpler-alerts-bot
# head_branch can be null. checking it here to see GITHUB_CONTEXT on master.
if: ${{ github.event.check_suite.head_branch == 'master' }}

0 comments on commit a8e2d73

Please sign in to comment.