Skip to content

Commit

Permalink
Merge pull request #66 from glensc/lint-checkstyle-cs2pr
Browse files Browse the repository at this point in the history
Use staabm/annotate-pull-request-from-checkstyle to report violatons in GitHub
  • Loading branch information
falkenhawk authored Apr 9, 2021
2 parents 616a12e + 5397451 commit dd57455
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ jobs:
with:
php-version: ${{ matrix.php }}
ini-values: display_errors=off, log_errors=on
tools: cs2pr
extensions: :xdebug, ctype, dom, gd, iconv, ldap, mbstring, reflection, session, simplexml, spl, xml, zlib, memcache, pdo_sqlite, bcmath
env:
# https://github.com/shivammathur/setup-php/issues/407#issuecomment-773675741
Expand All @@ -78,7 +79,7 @@ jobs:

- name: Lint PHP source files
run: |
composer lint
composer lint.checkstyle | cs2pr
- name: Setup environment for PHPUnit
run: |
Expand Down
6 changes: 4 additions & 2 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,10 @@
"symfony/polyfill-php70": "^1.19"
},
"require-dev": {
"php-parallel-lint/php-parallel-lint": "^1.0.0",
"php-parallel-lint/php-parallel-lint": "1.3.0",
"phpunit/dbunit": "1.3.2",
"zf1s/phpunit": "3.7.39"
"zf1s/phpunit": "3.7.39",
"staabm/annotate-pull-request-from-checkstyle": "1.5.0"
},
"autoload": {
"psr-0": {
Expand Down Expand Up @@ -206,6 +207,7 @@
},
"scripts": {
"lint": "parallel-lint --exclude vendor --exclude tests/Zend/Loader/_files/ParseError.php .",
"lint.checkstyle": "@lint --checkstyle",
"test": "phpunit"
}
}

0 comments on commit dd57455

Please sign in to comment.