Skip to content

Commit

Permalink
build: skip GitHub action steps the require write perms on external PRs
Browse files Browse the repository at this point in the history
... so that checks can pass when people submit PRs to the repo.
  • Loading branch information
djwhitt committed Aug 25, 2023
1 parent 44add3e commit be78dd0
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions .github/workflows/build-core.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,21 +39,24 @@ jobs:
# Test results report
- name: Report test results
uses: dorny/test-reporter@v1
if: success() || failure()
if: (success() || failure()) && ${{ github.event.pull_request.head.repo.full_name == github.repository }}
with:
name: Mocha tests
path: test-results.json
reporter: mocha-json

- name: Report code coverage
if: ${{ github.event_name == 'pull_request' }}
if: >
${{ github.event_name == 'pull_request' }} ||
(${{ github.event_name == 'pull_request' }} && ${{ github.event.pull_request.head.repo.full_name == github.repository }})
uses: romeovs/[email protected]
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
lcov-file: ./coverage/lcov.info

- name: Check code coverage threshold
uses: VeryGoodOpenSource/very_good_coverage@v2
if: ${{ github.event_name == 'pull_request' }} && ${{ github.event.pull_request.head.repo.full_name == github.repository }}
with:
path: ./coverage/lcov.info
min_coverage: 50
Expand All @@ -78,6 +81,7 @@ jobs:
# Build and push container image to ECR
- name: Configure AWS credentials
if: ${{ github.event_name != 'pull_request' }}
uses: aws-actions/configure-aws-credentials@v2
with:
role-to-assume: arn:aws:iam::${{ secrets.AWS_ACCOUNT_ID }}:role/${{ secrets.AWS_BUILD_INVOCATION_ROLE }}
Expand Down Expand Up @@ -107,15 +111,15 @@ jobs:
# Success notification
- name: Send Slack success notification
if: success()
if: success() && ${{ github.event_name != 'pull_request' }}
env:
SLACK_TITLE: Build succeeded!
SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }}
uses: rtCamp/action-slack-notify@v2

# Failure notification
- name: Send Slack failure notification
if: failure()
if: failure() && ${{ github.event_name != 'pull_request' }}
env:
SLACK_TITLE: Build failed!
SLACK_COLOR: danger
Expand Down

0 comments on commit be78dd0

Please sign in to comment.