Skip to content

Commit 9802787

Browse files
committed
ci: allow coverage upload to fail
Otherwise, PRs from (community) forks will never have green CI. In my opinion we can tolerate having no coverage info from the occasional community contributor as long as the tests are successful. Signed-off-by: Richard Steinmetz <[email protected]>
1 parent b780c9d commit 9802787

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

.github/workflows/npm-test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ jobs:
3737
token: ${{ secrets.CODECOV_TOKEN }}
3838
file: ./coverage/coverage-final.json
3939
flags: javascript
40-
fail_ci_if_error: true
40+
fail_ci_if_error: ${{ !github.event.pull_request.head.repo.fork }}
4141

4242
summary:
4343
runs-on: ubuntu-latest

.github/workflows/php-test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ jobs:
5050
token: ${{ secrets.CODECOV_TOKEN }}
5151
file: nextcloud/apps/calendar/clover.unit.xml
5252
flags: php
53-
fail_ci_if_error: true
53+
fail_ci_if_error: ${{ !github.event.pull_request.head.repo.fork }}
5454
verbose: true
5555

5656
integration-tests:

0 commit comments

Comments
 (0)