Skip to content

Conversation

@bohdanmoshchenko
Copy link
Collaborator

No description provided.

@b-vadym b-vadym marked this pull request as ready for review September 23, 2025 15:16
Copilot AI review requested due to automatic review settings September 23, 2025 15:16
@b-vadym b-vadym merged commit c7b8b74 into main Sep 23, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for PHP 8.4 by introducing a new PHP 8.4 rule set and updates all existing rule sets with improved type annotations and code organization. It also bumps vendor dependencies to their latest versions.

  • Introduces PHP 8.4 rule set with comprehensive coding standards
  • Refactors existing PHP rule sets to use imported classes instead of fully qualified names
  • Updates type annotations for better static analysis support

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/RuleSetInterface.php Enhanced type annotation for rules return type
src/RuleSet/Php84.php New PHP 8.4 rule set implementation
src/RuleSet/Php83.php Refactored imports and added Override attributes
src/RuleSet/Php82.php Refactored imports and added Override attributes
src/RuleSet/Php81.php Refactored imports and added Override attributes
src/RuleSet/Php80.php Refactored imports and added Override attributes
src/RuleSet/Php74.php Refactored imports and added Override attributes
src/Factory.php Enhanced type annotation for parameter
psalm.xml Added suppression for unused class warnings
composer.json Updated vendor dependencies to latest versions
README.md Updated documentation to reference PHP 8.4
.php-cs-fixer.dist.php Added explicit import for Finder class

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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