Skip to content

Conversation

@tomaj
Copy link
Owner

@tomaj tomaj commented Jul 23, 2025

Improve PHP 8+ compatibility and code quality by upgrading PHPStan and refactoring core classes.

This PR updates the composer.json to reflect PHP 7.4+/8.0+ compatibility and integrates phpstan/phpstan-strict-rules. Core classes (Message, Dispatcher, MessageSerializer) have been refactored to introduce typed properties, strict comparisons, and improved type handling, addressing issues identified by the stricter PHPStan analysis.


Open in WebOpen in Cursor

cursoragent and others added 2 commits July 23, 2025 17:43
Co-authored-by: tomasmajer <tomasmajer@gmail.com>
…n strict rules

Co-authored-by: tomasmajer <tomasmajer@gmail.com>
@cursor cursor bot force-pushed the cursor/skontroluj-spustenie-projektu-na-php-8-adcd branch from d98368f to b327650 Compare July 23, 2025 17:44
cursoragent and others added 9 commits July 23, 2025 17:44
Co-authored-by: tomasmajer <tomasmajer@gmail.com>
- Added information about enhanced PHPStan configuration
- Documented PHP 7.4+ minimum requirement and PHP 8.0+ support
- Listed core class improvements with typed properties
- Documented bug fixes and compatibility improvements
- Convert all properties to typed properties across all classes and traits
- Replace anonymous functions with arrow functions where applicable
- Use null coalescing operator (??) for cleaner null handling
- Fix strict comparisons for better type safety
- Improve error handling in SharedFileShutdown::shouldShutdown()
- Modernize all core classes: Emitter, Shutdown implementations, Traits
- Maintain backward compatibility while leveraging modern PHP features
- Remove PHP 7.2 and 7.3 from test matrix
- Add PHP 8.1, 8.2, and 8.3 to test matrix
- Update all GitHub Actions to latest versions (checkout@v4, cache@v3)
- Add proper PHP extensions setup (json, redis)
- Add composer validation and caching for faster builds
- Use PHP 8.1 as standard for static analysis tools
- Improve workflow names and job descriptions
- Add new security audit workflow with weekly schedule
- Enable workflows on pull requests for better CI coverage
- Document removal of PHP 7.2/7.3 support
- Add information about modern GitHub Actions setup
- Document new security audit workflow
- Include CI/CD improvements in changelog
- Delete tests/Handler/exception.log file
- File is already properly ignored in .gitignore
- Delete tests/Handler/exception--2025-07-23--17-29--29d60145dd.html
- File pattern is already ignored by .gitignore (exception--*)
- Delete tests/Handler/exception--2025-07-23--17-37--c884f00971.html
- Clean up test artifacts from repository
- Remove trailing whitespace from Message.php (lines 37-38)
- Add build directory creation in GitHub workflow
- Improve CodeClimate coverage upload with debug mode
- Add continue-on-error for coverage upload resilience
- Clean up .gitignore (remove duplicates, add trailing slashes)
- Add debug step to troubleshoot coverage file issues
@tomaj tomaj merged commit 31f8024 into master Jul 23, 2025
25 checks passed
@tomaj tomaj changed the title Skontroluj spustenie projektu na php 8+ Improve PHP 8+ compatibility and code quality by upgrading PHPStan and refactoring core classes. Jul 23, 2025
Sign up for free to 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.

3 participants