Skip to content

Commit 000712d

Browse files
ci: finetune when specific workflow steps shall run
1 parent 01903fa commit 000712d

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

.github/workflows/auto-request-review.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,11 @@ jobs:
1616
id: check_owner
1717
if: github.event.pull_request.user.id == 4849482 # martin-georgiev
1818
run: |
19+
echo "skip=true" >> $GITHUB_OUTPUT
1920
echo "Skipping auto-review request for repository owner"
20-
exit 0
2121
2222
- name: Request reviews based on configuration
23-
if: ${{ always() && steps.check_owner.outcome == 'skipped' }}
23+
if: ${{ !steps.check_owner.outputs.skip }}
2424
uses: necojackarc/auto-request-review@e89da1a8cd7c8c16d9de9c6e763290b6b0e3d424 # v0.13.0
2525
with:
2626
token: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ jobs:
101101
run: composer run-tests-with-clover
102102

103103
- name: Upload coverage results to Coveralls
104-
if: matrix.calculate-code-coverage == true
104+
if: matrix.calculate-code-coverage == true && github.event_name == 'push'
105105
uses: coverallsapp/github-action@648a8eb78e6d50909eff900e4ec85cab4524a45b # v2
106106
with:
107107
github-token: ${{ secrets.GITHUB_TOKEN }}

0 commit comments

Comments
 (0)