name: Static Code Analysis on: - pull_request - push permissions: contents: read # to fetch code (actions/checkout) concurrency: group: sca-${{ github.head_ref || github.run_id }} # will be canceled on subsequent pushes in pull requests but not branches cancel-in-progress: true jobs: tests: strategy: fail-fast: false matrix: operating-system: - ubuntu-20.04 php-version: - 8.1 name: Static Code Analysis runs-on: ${{ matrix.operating-system }} steps: - name: Checkout code uses: actions/checkout@v3 with: fetch-depth: 0 - name: Setup PHP uses: shivammathur/setup-php@v2 with: php-version: ${{ matrix.php-version }} extensions: none, dom, mbstring, opcache, simplexml, tokenizer, xml, xmlwriter coverage: none # without this Xdebug will be enabled tools: cs2pr - name: Get Composer cache directory id: composer-cache run: echo "dir=$(composer config cache-dir)" >> $GITHUB_OUTPUT - name: Cache dependencies uses: actions/cache@v3 with: path: ${{ steps.composer-cache.outputs.dir }} key: composer-${{ runner.os }}-${{ matrix.php-version }}-${{ hashFiles('**/composer.*') }}-${{ matrix.composer-flags }} restore-keys: | composer-${{ runner.os }}-${{ matrix.php-version }}-${{ hashFiles('**/composer.*') }}- composer-${{ runner.os }}-${{ matrix.php-version }}- composer-${{ runner.os }}- composer- - name: Install dependencies uses: nick-invision/retry@v2 with: timeout_minutes: 5 max_attempts: 5 retry_wait_seconds: 30 command: composer update --optimize-autoloader --no-interaction --no-progress ${{ matrix.composer-flags }} - name: Report versions run: composer info -D ## We want to have a lock-file used on PR level, so contributors are not bothered by SCA complains unrelated to their changes, ## and same time we want to be aware that we are complying with bleeding edge of SCA tools as maintainers observing the push hook. - name: Unlock dev-tools if: ${{ github.event_name != 'pull_request' }} run: rm ./dev-tools/composer.lock - name: Install dev-tools uses: nick-invision/retry@v2 with: timeout_minutes: 5 max_attempts: 5 retry_wait_seconds: 30 command: ./dev-tools/install.sh - name: Check - file permissions run: ./dev-tools/check_file_permissions.sh - name: Check - trailing spaces run: ./dev-tools/check_trailing_spaces.sh - name: Check - phpstan run: ./dev-tools/vendor/bin/phpstan analyse --error-format=checkstyle | cs2pr - name: Check - composer-require-checker run: ./dev-tools/vendor/bin/composer-require-checker check composer.json --config-file .composer-require-checker.json - name: Check - composer normalize run: | composer normalize --dry-run --working-dir=./dev-tools ../composer.json composer normalize --dry-run --working-dir=./dev-tools composer.json - name: Check - shell scripts run: ./dev-tools/check_shell_scripts.sh - name: Find changed files (for pull request) if: ${{ github.event_name == 'pull_request' }} run: | if git diff origin/$GITHUB_BASE_REF --name-only --diff-filter=ACMRTUXB | grep -E "\.php$"; then echo 'CHANGED_PHP_FILES<> $GITHUB_ENV git diff origin/$GITHUB_BASE_REF --name-only --diff-filter=ACMRTUXB | grep -E "\.php$" >> $GITHUB_ENV echo 'EOF' >> $GITHUB_ENV fi - name: Find changed files (for push) if: ${{ github.event_name != 'pull_request' }} run: | if git diff --name-only --diff-filter=ACMRTUXB HEAD~..HEAD | grep -E "\.php$"; then echo 'CHANGED_PHP_FILES<> $GITHUB_ENV git diff --name-only --diff-filter=ACMRTUXB HEAD~..HEAD | grep -E "\.php$" >> $GITHUB_ENV echo 'EOF' >> $GITHUB_ENV fi - name: Check - Mess Detector (phpmd) if: ${{ env.CHANGED_PHP_FILES }} run: | if [ '${{ github.event_name }}' == 'pull_request' ]; then ./dev-tools/vendor/bin/phpmd `echo "$CHANGED_PHP_FILES" | grep -v /Fixtures/ | xargs | sed 's/ /,/g'` github ./dev-tools/mess-detector/phpmd.xml else ./dev-tools/vendor/bin/phpmd `echo "$CHANGED_PHP_FILES" | grep -v /Fixtures/ | xargs | sed 's/ /,/g'` ansi ./dev-tools/mess-detector/phpmd.xml fi - name: Check - ensure test files are not present in the archive run: | git archive -o /dev/null HEAD -v 2>&1 | grep tests | grep \.php \ && (echo "Test files detected in archive" && exit 1) || echo "No test files detected in archive"