Skip to content

Commit

Permalink
Security-checker - fix testing security-checker in ci
Browse files Browse the repository at this point in the history
https://github.com/EdgedesignCZ/phpqa/pull/215/checks?check_run_id=1830905399
- Package enlightn/security-checker at version has a PHP requirement incompatible with your PHP version (5.4.45)

https://github.com/EdgedesignCZ/phpqa/pull/215/checks?check_run_id=1830905455
https://github.com/EdgedesignCZ/phpqa/runs/1838836954?check_suite_focus=true#step:5:130
- enlightn/security-checker v1.4 requires ext-zip * -> the requested PHP extension zip is missing from your system.

https://github.com/EdgedesignCZ/phpqa/pull/215/checks?check_run_id=1830905569#step:5:316
- Conclusion: don't install symfony/dependency-injection v2.8.50
- enlightn/security-checker v1.4 requires symfony/console ^3.4|^4|^5
  • Loading branch information
zdenekdrahos committed Feb 5, 2021
1 parent c04d5eb commit 752bbb0
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/php.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ jobs:
uses: shivammathur/setup-php@v2
with:
php-version: ${{ matrix.php }}
extensions: xsl
extensions: xsl, zip, json
coverage: none
# TODO: ? https://github.com/EdgedesignCZ/phpqa/runs/1813676801?check_suite_focus=true#step:6:28 ?
tools: composer:v1
Expand Down
6 changes: 5 additions & 1 deletion bin/suggested-tools.sh
Original file line number Diff line number Diff line change
Expand Up @@ -44,10 +44,14 @@ show_help() {
}

get_tools () {
TOOLS="php-parallel-lint/php-parallel-lint php-parallel-lint/php-console-highlighter enlightn/security-checker friendsofphp/php-cs-fixer:>=2"
TOOLS="php-parallel-lint/php-parallel-lint php-parallel-lint/php-console-highlighter friendsofphp/php-cs-fixer:>=2"
if [[ ${PHP_VERSION:0:1} != "5" ]]; then
TOOLS="${TOOLS} vimeo/psalm:>=2 phpstan/phpstan nette/neon"
fi
if [[ $UPDATE_COMPOSER == "yes" ]]; then
# security-checker does not support symfony2, so it cannot be installed on default composer.lock
TOOLS="${TOOLS} enlightn/security-checker"
fi
echo $TOOLS
}

Expand Down

0 comments on commit 752bbb0

Please sign in to comment.