Skip to content

Commit

Permalink
Add linting to Travis tests
Browse files Browse the repository at this point in the history
Protect against syntax compatibility issues such as the non-5.3
compatible array syntax corrected in 5785a9a by running a linter during
Travis CI testing for pull requests.

Closes #655
  • Loading branch information
bd808 committed Sep 27, 2015
1 parent ec0b383 commit 95d1370
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .travis.yml
Expand Up @@ -21,4 +21,4 @@ before_script:
- if [ "$deps" == "low" ]; then composer update --prefer-source --prefer-lowest --prefer-stable; fi
- if [ "$deps" != "low" ]; then composer install --prefer-source; fi

script: ./vendor/bin/phpunit
script: composer test
8 changes: 6 additions & 2 deletions composer.json
Expand Up @@ -26,7 +26,8 @@
"videlalvaro/php-amqplib": "~2.4",
"swiftmailer/swiftmailer": "~5.3",
"php-console/php-console": "^3.1.3",
"phpunit/phpunit-mock-objects": "2.3.0"
"phpunit/phpunit-mock-objects": "2.3.0",
"jakub-onderka/php-parallel-lint": "0.9"
},
"_": "phpunit/phpunit-mock-objects required in 2.3.0 due to https://github.com/sebastianbergmann/phpunit-mock-objects/issues/223 - needs hhvm 3.8+ on travis",
"suggest": {
Expand Down Expand Up @@ -56,6 +57,9 @@
}
},
"scripts": {
"test": "phpunit"
"test": [
"parallel-lint . --exclude vendor",
"phpunit"
]
}
}

0 comments on commit 95d1370

Please sign in to comment.