Updated tests in order to be executed in PHP 8.4, and supports Php 8.4

Author: Halleck45

.github/FUNDING.yml

@@ -0,0 +1,3 @@
+# These are supported funding model platforms
+
+github: [Halleck45]

.github/workflows/ci.yml

@@ -0,0 +1,76 @@
+name: CI
+
+on:
+    push:
+        branches: [ master, main ]
+    pull_request:
+        branches: [ master, main ]
+    release:
+        types: [created]
+
+jobs:
+    test:
+        runs-on: ubuntu-latest
+        strategy:
+            matrix:
+                php-version: [7.0, 7.1, 7.2, 7.3, 7.4, 8.0, 8.1, 8.2, 8.3, 8.4]
+                #php-version: [5.6, 7.0, 7.1, 7.2, 7.3, 7.4, 8.0, 8.1, 8.2, 8.3, 8.4]
+                include:
+                    - php-version: nightly
+            fail-fast: false
+
+        services:
+            docker:
+                image: docker:20.10.16
+                options: --privileged
+
+        steps:
+            - uses: actions/checkout@v4
+
+            - name: Set up PHP ${{ matrix.php-version }}
+              uses: shivammathur/setup-php@v2
+              with:
+                  php-version: ${{ matrix.php-version }}
+
+            - name: "Remove ': void' return type hint for older PHP versions"
+              run: |
+                # if php version < 7.1, remove ': void' return type hint
+                if [[ "${{ matrix.php-version }}" < "7.1" ]]; then
+                    echo "Removing ': void' return type hint for PHP version ${{ matrix.php-version }}"
+                    # Find and replace ': void' in all PHP files
+                    find . -type f -name "*.php" -exec sed -i 's/: void//g' {} +
+                fi
+
+            - name: Install Composer dependencies (${{ matrix.php-version }})
+              run: composer install --no-interaction --prefer-dist --no-progress
+
+            - name: Run tests (${{ matrix.php-version }})
+              run: make test
+
+            - name: Run Docker tests
+              run: docker run --rm test_phpmetrics
+
+    build:
+        if: github.event_name == 'release' && startsWith(github.ref, 'refs/tags/')
+        needs: test
+        runs-on: ubuntu-latest
+        steps:
+            -   uses: actions/checkout@v4
+
+            -   name: Set up PHP
+                uses: shivammathur/setup-php@v2
+                with:
+                    php-version: 7.0
+
+            -   name: Build project
+                run: make build
+
+            -   name: Add build artifact
+                run: git add -f build/phpmetrics.phar
+
+            -   name: Upload release asset
+                uses: softprops/action-gh-release@v2
+                with:
+                    files: build/phpmetrics.phar
+                env:
+                    GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.gitignore

@@ -1,5 +1,6 @@
 composer.lock
 vendor
+tooling/vendor
 .idea
 md5sums
 control
@@ -8,3 +9,12 @@ build/phpmetrics.dsc
 build/phpmetrics.deb
 build/phpmetrics.tar.gz
 build/phpmetrics.changes
+.php-cs-fixer.cache
+.phpunit.result.cache
+
+# build
+buildroot/
+downloads/
+pkgroot/
+source/
+spc

.php-cs-fixer.php

@@ -0,0 +1,14 @@
+<?php
+
+$finder = (new PhpCsFixer\Finder())
+    ->in(__DIR__ . '/src')
+;
+
+return (new PhpCsFixer\Config())
+    ->setRules([
+        '@PER-CS' => true,
+        '@PHP82Migration' => true,
+    ])
+    ->setFinder($finder)
+    ->setUnsupportedPhpVersionAllowed(true)
+    ;

Makefile

@@ -2,17 +2,17 @@
 
 include artifacts/Makefile
 
-# Run unit tests
+# Run unit tests
 test:
 	./vendor/bin/phpunit -c phpunit.xml.dist
 
 # Codesniffer check
 phpcs:
-	./vendor/bin/phpcs src/ tests/ --extensions=php -n
+	./tooling/vendor/bin/php-cs-fixer check src
 
-# Codesniffer fix
-phpcbf:
-	./vendor/bin/phpcbf src/ tests/ --extensions=php -n
+# Compatibility check
+compatibility:
+	(docker run --rm -v `pwd`:/www --workdir=/www  php:5.6-cli find src -iname "*.php" -exec php -l {} \; |grep -v "Php7NodeTraverser.php" | grep -v "No syntax errors detected") && echo OK
 
 # Used for tag releasing
 # Don't use directly, use `make release` instead

artifacts/Makefile

@@ -4,6 +4,7 @@ BUILD_DIR=releases
 
 include artifacts/phar/Makefile
 include artifacts/debian/Makefile
+include artifacts/standalone/Makefile
 
 prepare-build:
 	@# Disable the deletion of all releases packages as task build-debian is disabled.
@@ -12,5 +13,5 @@ prepare-build:
 	@# Only remove the phar that must be replaced by the new release.
 	@rm -f ${BUILD_DIR}/phpmetrics.phar
 
-build: prepare-build build-phar build-deb
+build: prepare-build build-phar build-deb build-standalone
 

composer.json

@@ -30,22 +30,23 @@
         },
         "files": ["./src/functions.php"]
     },
+    "autoload-dev": {
+        "psr-4": {
+            "Test\\Hal\\": "tests/"
+        }
+    },
     "require": {
-        "php": ">=5.5",
         "ext-dom": "*",
         "ext-tokenizer": "*",
-        "nikic/php-parser": "^3|^4"
-    },
-    "require-dev": {
-        "phpunit/phpunit": "^4.8.36 || ^5.7.27 || ^6.5.14",
-        "sebastian/comparator": ">=1.2.3",
-        "squizlabs/php_codesniffer": "^3.5",
-        "symfony/dom-crawler": "^3.0 || ^4.0 || ^5.0"
+        "nikic/php-parser": "^3|^4|^5"
     },
     "bin": [
         "bin/phpmetrics"
     ],
     "config": {
         "sort-packages": true
+    },
+    "require-dev": {
+        "phpunit/phpunit": "*"
     }
 }

craft.yml

@@ -0,0 +1,6 @@
+php-version: 8.4
+extensions: "apcu,phar,curl,dom,fileinfo,filter,intl,mbstring,mysqlnd,openssl,tokenizer,zlib"
+sapi: micro
+build-options:
+    with-upx-pack: true
+    prefer-pre-built: true

doc/contributing.md

@@ -11,6 +11,27 @@ Then, in order to run the test suite:
 
 Thanks for your help.
 
+## Why the code is so old?
+
+### Philosophy
+
+PhpMetrics has several goals:
++ be stable
++ be performant
++ run on the **maximum of PHP versions** (PHP 5.3 to PHP 8.4)
++ be installable everywhere, **without dependency conflicts**
+
+For these reasons, the code of PhpMetrics is intentionally written in "legacy" PHP.
+
+### Dependencies
+
+Not all projects are on the latest version of PHP, Symfony, or Laravel. Yes, there are projects that use Symfony 2. And these projects may also need metrics and quality.
+
+For these reasons, PhpMetrics comes with the minimum of dependencies. Only the dependency on `nikic/php-parser` is accepted.
+
+No Pull Request that modifies the `require` block will be accepted.
+
+
 ## Releasing
 
 Please NEVER tag manually.