Skip to content

Commit 389f9a6

Browse files
authored
Merge pull request #5271 from LibreSign/fix/previous-pr
fix: move back to be a step
2 parents 0c0414a + 1ec2707 commit 389f9a6

1 file changed

Lines changed: 8 additions & 14 deletions

File tree

.github/workflows/phpunit-mysql.yml

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,14 @@ jobs:
169169
working-directory: apps/${{ env.APP_NAME }}
170170
run: composer run test:coverage
171171

172+
- name: Upload coverage results to Coveralls
173+
working-directory: apps/${{ env.APP_NAME }}
174+
env:
175+
COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
176+
COVERALLS_PARALLEL: true
177+
COVERALLS_FLAG_NAME: php-${{ matrix.php-versions }}
178+
run: vendor/bin/php-coveralls --coverage_clover=build/logs/clover.xml --json_path=build/logs/coveralls-upload.json -v
179+
172180
- name: Check PHPUnit integration script is defined
173181
id: check_integration
174182
continue-on-error: true
@@ -213,20 +221,6 @@ jobs:
213221
- name: Summary status
214222
run: if ${{ needs.changes.outputs.src != 'false' && needs.phpunit-mysql.result != 'success' }}; then exit 1; fi
215223

216-
upload-coverage:
217-
needs: [phpunit-mysql, matrix]
218-
runs-on: ubuntu-latest
219-
strategy:
220-
matrix:
221-
php-versions: ${{ fromJson(needs.matrix.outputs.php-version) }}
222-
steps:
223-
- name: Upload coverage results to Coveralls
224-
working-directory: apps/${{ env.APP_NAME }}
225-
env:
226-
COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
227-
COVERALLS_PARALLEL: true
228-
COVERALLS_FLAG_NAME: php-${{ matrix.php-versions }}
229-
run: vendor/bin/php-coveralls --coverage_clover=build/logs/clover.xml --json_path=build/logs/coveralls-upload.json -v
230224
finish-coverage:
231225
needs: phpunit-mysql
232226
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)