Skip to content

Commit 70d8120

Browse files
authored
Fix dependencies (#34)
* Fix dependencies * Put code coverage in a different action * Fix matrix * Modify scrutinizer config
1 parent e4dd199 commit 70d8120

File tree

3 files changed

+24
-23
lines changed

3 files changed

+24
-23
lines changed

.github/workflows/code_checks.yml renamed to .github/workflows/validate.yml

Lines changed: 9 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -2,22 +2,22 @@ name: PHP Composer
22

33
on:
44
push:
5-
branches: [ master ]
5+
branches:
6+
- 'master'
67
pull_request:
7-
branches: [ master ]
8+
branches:
9+
- 'master'
810

911
jobs:
1012
build:
11-
13+
name: PHP ${{ matrix.php }} - Laravel ${{ matrix.laravel }}
1214
runs-on: ubuntu-latest
1315
strategy:
1416
fail-fast: true
1517
matrix:
1618
php: [ '7.2', '7.3', '7.4' ]
1719
laravel: [ '5.5.*', '5.6.*', '5.7.*', '5.8.*', '^6.0', '^7.0' ]
1820

19-
name: PHP ${{ matrix.php }} - Laravel ${{ matrix.laravel }}
20-
2121
steps:
2222
- uses: actions/checkout@v2
2323

@@ -29,20 +29,15 @@ jobs:
2929
- uses: shivammathur/setup-php@v1
3030
with:
3131
php-version: ${{ matrix.php }}
32-
coverage: xdebug
32+
coverage: none
3333

34-
- name: Validate composer.json and composer.lock
34+
- name: Validate composer.json
3535
run: composer validate
3636

3737
- name: Install dependencies
3838
run: |
39-
composer require "illuminate/support:${{ matrix.laravel }}" "illuminate/support:${{ matrix.laravel }}" --no-interaction --no-update
39+
composer require "illuminate/support:${{ matrix.laravel }}" "illuminate/http:${{ matrix.laravel }}" --no-interaction --no-update
4040
composer update --prefer-dist --no-interaction --no-suggest
4141
4242
- name: Run test
43-
run: ./vendor/bin/phpunit --coverage-clover=coverage.clover --verbose
44-
45-
- name: External Code Coverage
46-
run: |
47-
wget https://scrutinizer-ci.com/ocular.phar
48-
php ocular.phar code-coverage:upload --format=php-clover coverage.clover
43+
run: ./vendor/bin/phpunit --verbose

.scrutinizer.yml

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,20 @@
11
filter:
22
paths:
3-
- 'src/*'
3+
- 'src/'
44
excluded_paths:
5-
- 'tests/*'
6-
- 'vendor/*'
5+
- 'tests/'
6+
dependency_paths:
7+
- 'vendor/'
78

89
checks:
910
php: true
1011

11-
tools:
12-
external_code_coverage:
13-
runs: 3
14-
timeout: 1200
12+
build:
13+
nodes:
14+
coverage:
15+
tests:
16+
override:
17+
- command: ./vendor/bin/phpunit --coverage-clover=coverage.clover
18+
coverage:
19+
file: coverage.clover
20+
format: clover

composer.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,8 @@
1919
],
2020
"require": {
2121
"php": "^7.1",
22-
"illuminate/support": "^5.5 || ^6.0",
23-
"illuminate/http": "^5.5 || ^6.0"
22+
"illuminate/support": "^5.5 || ^6.0 || ^7.0",
23+
"illuminate/http": "^5.5 || ^6.0 || ^7.0"
2424
},
2525
"require-dev": {
2626
"orchestra/testbench": "^3.5 || ^4.0 || ^5.0",

0 commit comments

Comments
 (0)