-
Notifications
You must be signed in to change notification settings - Fork 6
Support PHP 8.5 #335
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support PHP 8.5 #335
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -14,7 +14,7 @@ jobs: | |||||
| runs-on: ubuntu-24.04 | ||||||
| strategy: | ||||||
| matrix: | ||||||
| php-versions: ['7.3', '7.4', '8.0'] | ||||||
| php-versions: ['8.1', '8.2', '8.3', '8.4', '8.5'] | ||||||
|
|
||||||
| steps: | ||||||
| - uses: actions/checkout@v4 | ||||||
|
|
@@ -40,49 +40,10 @@ jobs: | |||||
| run: composer update --prefer-dist --no-interaction | ||||||
|
|
||||||
| - name: Run phpunit | ||||||
| run: XDEBUG_MODE=coverage vendor/bin/phpunit --coverage-text --configuration=phpunit-ci-old-versions.xml | ||||||
| run: XDEBUG_MODE=coverage vendor/bin/phpunit --color="always" --coverage-text --configuration=phpunit-ci.xml | ||||||
|
|
||||||
| - name: Install lowest dependencies | ||||||
| run: composer update --prefer-dist --no-interaction --prefer-lowest | ||||||
|
|
||||||
| - name: Run phpunit with lowest dependencies | ||||||
| run: XDEBUG_MODE=coverage vendor/bin/phpunit --coverage-text --configuration=phpunit-ci-old-versions.xml | ||||||
|
|
||||||
| build-new-php-version: | ||||||
|
|
||||||
| services: | ||||||
| rabbitmq: | ||||||
| image: rabbitmq:4.1 | ||||||
| ports: | ||||||
| - 5672:5672 | ||||||
|
|
||||||
| runs-on: ubuntu-24.04 | ||||||
| strategy: | ||||||
| matrix: | ||||||
| php-versions: ['8.1', '8.2', '8.3', '8.4', '8.5'] | ||||||
|
|
||||||
| steps: | ||||||
| - uses: actions/checkout@v4 | ||||||
|
|
||||||
| - name: Setup PHP | ||||||
| uses: shivammathur/setup-php@v2 | ||||||
| with: | ||||||
| php-version: ${{ matrix.php-versions }} | ||||||
| extensions: sockets, xdebug | ||||||
| coverage: xdebug | ||||||
| tools: phpunit:latest | ||||||
|
|
||||||
| - name: Cache Composer packages | ||||||
| id: composer-cache | ||||||
| uses: actions/cache@v4 | ||||||
| with: | ||||||
| path: vendor | ||||||
| key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }} | ||||||
| restore-keys: | | ||||||
| ${{ runner.os }}-php- | ||||||
|
|
||||||
| - name: Install dependencies | ||||||
| run: composer update --prefer-dist --no-interaction | ||||||
|
|
||||||
| - name: Run phpunit | ||||||
| run: XDEBUG_MODE=coverage vendor/bin/phpunit --color="always" --coverage-text --configuration=phpunit-ci.xml | ||||||
| run: XDEBUG_MODE=coverage vendor/bin/phpunit --coverage-text --configuration=phpunit-ci.xml | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Missing newline at end of file. This can cause issues with some tools and violates POSIX standards.
Suggested change
|
||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -9,6 +9,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 | |||||
| ### Added | ||||||
| - Support PHP 8.5 | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛑 Premature Version Claim: Claiming PHP 8.5 support is misleading since PHP 8.5 is not yet released (scheduled for November 2025). This could confuse users about actual compatibility.
Suggested change
|
||||||
|
|
||||||
| ### Removed | ||||||
| - Support PHP 7.3, 7.4, 8.0 | ||||||
| - Support Laravel ~6.0, ~7.0 | ||||||
|
|
||||||
| ## [v2.7.0] | ||||||
| ### Added | ||||||
| - Support for RabbitMQ 4.x.x | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -17,17 +17,17 @@ | |||||
| } | ||||||
| ], | ||||||
| "require": { | ||||||
| "php": ">=7.3", | ||||||
| "php": ">=8.1", | ||||||
| "ext-json": "*", | ||||||
| "ext-sockets": "*", | ||||||
| "php-amqplib/php-amqplib": "^3.1", | ||||||
| "laravel/framework": "^6.20.12|^7.30.4|^8.22.1|^9.2|^10.0|^11.0|^12.0" | ||||||
| "laravel/framework": "^8.22.1|^9.2|^10.0|^11.0|^12.0" | ||||||
| }, | ||||||
| "require-dev": { | ||||||
| "ext-curl": "*", | ||||||
| "phpunit/phpunit": "^9.5|^10.0|^11.0", | ||||||
| "phpunit/phpunit": "^10.0|^11.0", | ||||||
| "squizlabs/php_codesniffer": "^2.5", | ||||||
| "orchestra/testbench": "^4.18|^6.21|^8.0|^9.0|^10.0", | ||||||
| "orchestra/testbench": "^6.21|^8.0|^9.0|^10.0", | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛑 Compatibility Gap: Missing orchestra/testbench version ^7.0 creates a gap between ^6.21 and ^8.0, potentially causing dependency resolution issues for Laravel 8.x projects.
Suggested change
|
||||||
| "mockery/mockery": "^1.3" | ||||||
| }, | ||||||
| "autoload": { | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -5,10 +5,16 @@ | |||||
| 3. [Configuring Pigeon](#configuring) | ||||||
|
|
||||||
| ## Requirements | ||||||
| - PHP >= 7.1.3 | ||||||
| - JSON PHP extension | ||||||
| - Sockets PHP Extension | ||||||
|
|
||||||
| ## Compatibility | ||||||
| | Package Version | PHP Compatibility | Laravel Compatibility | | ||||||
| |-----------------|-------------------|------------------------| | ||||||
| | Version 1 | >= 7.1.3 | 5.6, 6.x, 7.x, 8.x | | ||||||
| | Version 2 | >= 7.3 | 6.20.12+, 7.30.4+, 8.22.1+, 9.x, 10.x, 11.x, 12.x | | ||||||
| | Version 3 | >= 8.1 | 8.22.1+, 9.x, 10.x, 11.x, 12.x | | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛑 Version Inconsistency: The documentation shows "Version 3" for PHP >= 8.1, but the changelog indicates this is v2.8.0. This creates confusion about the actual version scheme and could mislead users about compatibility.
Suggested change
|
||||||
|
|
||||||
| ## Installing | ||||||
| Pigeon utilizes [Composer](https://getcomposer.org/) to manage its dependencies. So, before using Laravel, make sure you have Composer installed on your machine. | ||||||
|
|
||||||
|
|
||||||
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛑 Premature Version Support: PHP 8.5 is not yet released and may cause CI failures. PHP 8.5 is scheduled for release in November 2025, but this workflow will fail until then because the version doesn't exist in setup-php action.