Skip to content

Preparation for PHP 8.4: Rector rules and CI compatibility check - #19

Merged
agissept merged 6 commits into
masterfrom
feature/preparation-php84
Jun 18, 2026
Merged

Preparation for PHP 8.4: Rector rules and CI compatibility check#19
agissept merged 6 commits into
masterfrom
feature/preparation-php84

Conversation

@agissept

@agisseptagissept commented May 12, 2026

Copy link
Copy Markdown
Member

Summary

  • Add PHP 8.4 compatibility CI job using Rector --dry-run on changed files only
  • Set withPhpVersion(PhpVersion::PHP_84) in rector.php
  • Add Rector rules for PHP 8.4: ExplicitNullableParamTypeRector, AddEscapeArgumentRector
  • Apply existing PHP 8.3/8.4 Rector rules across codebase (deprecated annotations → attributes, explicit nullable params, escape arguments, etc.)
  • Fix IronQueue::recreate(): optional parameter $queue before required $delay — deprecated in PHP 8.4
  • Update predis and portable-ascii to PHP 8.4-compatible versions

Changed files

AreaDescription
.github/workflows/pull-request-check.ymlNew CI job: Rector dry-run on changed PHP files with PHP 8.4
rector.phpAdd PHP 8.4 rules and version target
composer.jsonUpdate predis & portable-ascii
src/Illuminate/** (34 files)Apply Rector rules

Test plan

  • CI php84-syntax-check job passes on this PR
  • Existing unit tests pass
  • Verify deprecated annotations replaced with #[Deprecated] attribute
  • Verify implicit nullable params replaced with explicit ?Type

🤖 Generated with Claude Code

@agissept
agisseptforce-pushed the feature/preparation-php84 branch from 7223225 to 6c843fcCompareMay 12, 2026 09:42
agisseptand others added 6 commits May 12, 2026 17:04
apply rule ExplicitNullableParamTypeRector
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- IronQueue::recreate: add default value to $delay to fix deprecated
optional-before-required parameter order in PHP 8.4
- Simplify syntax check CI step so php -l errors print directly to log
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@agissept
agisseptforce-pushed the feature/preparation-php84 branch 3 times, most recently from 4252897 to 76f6076CompareMay 12, 2026 10:31
@agisseptagissept changed the title PHP 8.4 compatibility: Rector rules and dependency updatesPreparation for PHP 8.4: Rector rules and CI compatibility checkMay 12, 2026
@agisseptagissept self-assigned this Jun 18, 2026
@agissept
agissept merged commit 1d39885 into masterJun 18, 2026
2 checks passed
@agissept
agissept deleted the feature/preparation-php84 branch June 18, 2026 01:47
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@agissept