Merge pull request #1 from tithely/php-8

MEM-151 support php-8.4

Author: alankoroma

.github/workflows/test.yaml

@@ -1,31 +1,25 @@
 name: Test
 
-on: [push, pull_request]
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: '*'
 
 jobs:
     test:
-        name: PHP ${{ matrix.php-version }} (${{ matrix.experimental && 'experimental' || 'full support' }})
+        name: PHP ${{ matrix.php-version }}
 
-        runs-on: ubuntu-18.04
+        runs-on: ubuntu-latest
+        timeout-minutes: 10
 
         strategy:
-            fail-fast: false
             matrix:
                 php-version:
-                    - 7.1
-                    - 7.2
-                    - 7.3
-                    - 7.4
-                    - 8.0
-                experimental: [false]
-                include:
-                    - php-version: 8.1
-                      experimental: true
-
-        continue-on-error: ${{ matrix.experimental }}
+                    - 8.4
         steps:
             - name: Checkout
-              uses: actions/checkout@v2
+              uses: actions/checkout@v5
 
             - name: Install PHP with extensions
               uses: shivammathur/setup-php@v2
@@ -34,11 +28,21 @@ jobs:
                   coverage: pcov
                   tools: composer:v2
 
-            - name: Install Composer dependencies
-              uses: ramsey/composer-install@v1
+            - name: Validate composer.json and composer.lock
+              run: composer validate
+
+            - name: Cache Composer packages
+              id: composer-cache
+              uses: actions/cache@v3
               with:
-                  composer-options: --prefer-dist
-              continue-on-error: ${{ matrix.experimental }}
+                path: vendor
+                key: ${{ runner.os }}-${{ matrix.php-versions }}-composer-${{ hashFiles('**/composer.json') }}
+                restore-keys: |
+                  ${{ runner.os }}-${{ matrix.php-versions }}-composer
+
+            - name: Install dependencies
+              if: steps.composer-cache.outputs.cache-hit != 'true'
+              run: composer install --prefer-dist --no-progress
 
             - name: Setup PCOV
               run: |
@@ -47,9 +51,4 @@ jobs:
               continue-on-error: true
 
             - name: Run Tests
-              run: composer tests
-              continue-on-error: ${{ matrix.experimental }}
-
-            - name: Check coding style
-              run: composer coding-style
-              continue-on-error: ${{ matrix.experimental }}
+              run: composer test

.gitignore

@@ -6,3 +6,4 @@ phpcs.xml
 phpunit.xml
 .idea/
 .phpunit.result.cache
+.php-cs-fixer.cache

.php_cs.dist .php-cs-fixer.dist.php.php_cs.dist renamed to .php-cs-fixer.dist.php

@@ -1,6 +1,6 @@
 <?php
 
-return PhpCsFixer\Config::create()
+return (new PhpCsFixer\Config())
     ->setRiskyAllowed(true)
     ->setRules([
         'native_function_invocation' => true,

composer.json

@@ -1,5 +1,5 @@
 {
-    "name": "lstrojny/functional-php",
+    "name": "tithely/functional-php",
     "description": "Functional primitives for PHP",
     "keywords": [
         "functional"
@@ -17,12 +17,12 @@
         }
     ],
     "require": {
-        "php": "^7.1|~8"
+        "php": "^8.4"
     },
     "require-dev": {
         "squizlabs/php_codesniffer": "~3.0",
-        "phpunit/phpunit": "^7.0 || ^8.0 || ^9.5",
-        "friendsofphp/php-cs-fixer": "^2.17"
+        "phpunit/phpunit": "^9.6",
+        "friendsofphp/php-cs-fixer": "v3.84"
     },
     "autoload": {
         "psr-4": {
@@ -132,8 +132,16 @@
         }
     },
     "scripts": {
-        "tests": "vendor/bin/phpunit",
-        "coding-style": "vendor/bin/phpcs && vendor/bin/php-cs-fixer fix --dry-run --diff --config=.php_cs.dist",
-        "clear": "rm -rf vendor/"
+        "test": "vendor/bin/phpunit",
+        "test:84": "docker exec -w /app functional_php_84 ./vendor/bin/phpunit --colors=always --no-coverage",
+        "lint:scan": "vendor/bin/php-cs-fixer fix src --config=.php-cs-fixer.dist.php --dry-run",
+        "lint:fix": "vendor/bin/php-cs-fixer fix src --config=.php-cs-fixer.dist.php",
+        "clear": "rm -rf vendor/",
+        "clean-vendor": "rm -rf vendor && rm -rf composer.lock",
+        "docker:start": "docker compose -p functional_php -f docker/docker-compose.yml up -d",
+        "docker:stop": "docker compose -p functional_php -f docker/docker-compose.yml down",
+        "docker:rebuild": "docker compose -p functional_php -f docker/docker-compose.yml build --force-rm && docker-compose -p functional_php -f docker/docker-compose.yml up -d --remove-orphans",
+        "docker:rebuild:nocache": "docker compose -p functional_php -f docker/docker-compose.yml build --force-rm --no-cache && docker-compose -p functional_php -f docker/docker-compose.yml up -d --remove-orphans",
+        "docker:shell": "docker exec -w /app -it functional_php_84 /bin/bash"
     }
 }

docker/custom.ini

@@ -0,0 +1,2 @@
+post_max_size = 100M
+max_execution_time = 300

docker/docker-compose.yml

@@ -0,0 +1,17 @@
+services:
+  functional_php_84:
+    build:
+      dockerfile: docker/php/84.Dockerfile
+      context: ../
+    container_name: functional_php_84
+    volumes:
+      - ../:/app
+    networks:
+      - tithely_hub_code-network
+    environment:
+      - CONF_DISABLED=xdebug
+      - PCOV_ENABLED=1
+      - PCOV_DIRECTORY=/app
+networks:
+    tithely_hub_code-network:
+        external: true

docker/php/84.Dockerfile

@@ -0,0 +1,13 @@
+FROM tithely/php:8.4-apache-dev
+
+#------------------------------
+# Begin changes below this note
+#------------------------------
+
+# Install custom packages
+RUN apt-get update -qq && \
+    apt-get install -qq \
+    vim
+
+# Copy custom php.ini settings
+COPY docker/custom.ini /usr/local/etc/php/conf.d/custom_overrides.ini

src/Functional/CompareObjectHashOn.php

@@ -14,11 +14,11 @@
  * Returns a comparison function that can be used with e.g. `usort()`
  *
  * @param callable $comparison A function that compares the two values. Pick e.g. strcmp() or strnatcasecmp()
- * @param callable $keyFunction A function that takes an argument and returns the value that should be compared
+ * @param callable|null $keyFunction A function that takes an argument and returns the value that should be compared
  * @return callable
  * @no-named-arguments
  */
-function compare_object_hash_on(callable $comparison, callable $keyFunction = null)
+function compare_object_hash_on(callable $comparison, ?callable $keyFunction = null)
 {
     $keyFunction = $keyFunction ? compose($keyFunction, 'spl_object_hash') : 'spl_object_hash';
 

src/Functional/CompareOn.php

@@ -14,11 +14,11 @@
  * Returns a comparison function that can be used with e.g. `usort()`
  *
  * @param callable $comparison A function that compares the two values. Pick e.g. strcmp() or strnatcasecmp()
- * @param callable $reducer A function that takes an argument and returns the value that should be compared
+ * @param callable|null $reducer A function that takes an argument and returns the value that should be compared
  * @return callable
  * @no-named-arguments
  */
-function compare_on(callable $comparison, callable $reducer = null)
+function compare_on(callable $comparison, ?callable $reducer = null)
 {
     if ($reducer === null) {
         return static function ($left, $right) use ($comparison) {

src/Functional/Concat.php

@@ -13,7 +13,7 @@
 /**
  * Concatenates zero or more strings
  *
- * @param string[] ...$strings
+ * @param string ...$strings
  * @return string
  * @no-named-arguments
  */