Skip to content

Commit

Permalink
Merge pull request #118 from mfn/mfn-l8
Browse files Browse the repository at this point in the history
Add explicit support for Laravel 8
  • Loading branch information
antonioribeiro authored Sep 20, 2020
2 parents 1c8f323 + ce96d7c commit d6505cb
Show file tree
Hide file tree
Showing 3 changed files with 29 additions and 12 deletions.
28 changes: 21 additions & 7 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,28 +4,42 @@ php:
- 7.1
- 7.2
- 7.3
- 7.4snapshot
- 7.4
# - nightly

# This triggers builds to run on the new TravisCI infrastructure.
# See: http://docs.travis-ci.com/user/workers/container-based-infrastructure/
sudo: false

## Cache composer
cache:
directories:
- $HOME/.composer/cache

env:
- LARAVEL_VERSION=5.8.* TESTBENCH_VERSION=3.8.*
- LARAVEL_VERSION=6.* TESTBENCH_VERSION=4.*
- LARAVEL_VERSION=7.* TESTBENCH_VERSION=5.*
- LARAVEL_VERSION=8.* TESTBENCH_VERSION=6.*

matrix:
exclude:
- php: 7.1
env: LARAVEL_VERSION=6.* TESTBENCH_VERSION=4.*
- php: 7.1
env: LARAVEL_VERSION=7.* TESTBENCH_VERSION=5.*
- php: 7.1
env: LARAVEL_VERSION=8.* TESTBENCH_VERSION=6.*
- php: 7.2
env: LARAVEL_VERSION=8.* TESTBENCH_VERSION=6.*

before_script:
- yes '' | pecl install imagick
- travis_retry composer update ${COMPOSER_FLAGS} --no-interaction --prefer-dist
- composer require "laravel/framework:${LARAVEL_VERSION}" "orchestra/testbench:${TESTBENCH_VERSION}" --no-update
- travis_retry composer update --no-interaction --prefer-dist

script:
- vendor/bin/phpunit --coverage-text --coverage-clover=coverage.clover

after_script:
- |
if [[ "$TRAVIS_PHP_VERSION" == '7.3' ]]; then
if [[ "$TRAVIS_PHP_VERSION" == '7.4' ]]; then
wget https://scrutinizer-ci.com/ocular.phar
php ocular.phar code-coverage:upload --format=php-clover coverage.clover
fi
8 changes: 5 additions & 3 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,12 @@
],
"require": {
"php": ">=7.0",
"laravel/framework": ">=5.4.36",
"laravel/framework": ">=5.4.36|^8.0",
"pragmarx/google2fa-qrcode": "^1.0"
},
"require-dev": {
"phpunit/phpunit": "~5|~6|~7|~8",
"orchestra/testbench": "3.4.*|3.5.*|3.6.*|3.7.*|4.*"
"orchestra/testbench": "3.4.*|3.5.*|3.6.*|3.7.*|4.*|5.*|6.*"
},
"autoload": {
"psr-4": {
Expand All @@ -43,5 +43,7 @@
"suggest": {
"bacon/bacon-qr-code": "Required to generate inline QR Codes.",
"pragmarx/recovery": "Generate recovery codes."
}
},
"minimum-stability": "dev",
"prefer-stable": true
}
5 changes: 3 additions & 2 deletions tests/Support/User.php
Original file line number Diff line number Diff line change
Expand Up @@ -60,11 +60,12 @@ public function getQueueableConnection()
/**
* Retrieve the model for a bound value.
*
* @param mixed $value
* @param mixed $value
* @param string|null $field
*
* @return \Illuminate\Database\Eloquent\Model|null
*/
public function resolveRouteBinding($value)
public function resolveRouteBinding($value, $field = null)
{
// TODO: Implement resolveRouteBinding() method.
}
Expand Down

0 comments on commit d6505cb

Please sign in to comment.