Skip to content

fix all phpstan l6 errors#688

Open
fadrian06 wants to merge 2 commits intomasterfrom
fix-phpstan-errors
Open

fix all phpstan l6 errors#688
fadrian06 wants to merge 2 commits intomasterfrom
fix-phpstan-errors

Conversation

@fadrian06
Copy link
Contributor

This pull request primarily improves code readability and consistency by reformatting long lines for better clarity and updating PHPDoc type annotations for better type accuracy. Additionally, it updates the static analysis configuration to refine which files are included in analysis.

Code formatting and readability improvements:

  • Reformatted long strings in user messages and function calls in flight/commands/RouteCommand.php and flight/commands/AiGenerateInstructionsCommand.php to improve readability and maintainability. [1] [2] [3]

Type annotation updates:

  • Updated PHPDoc type annotations in constructors of PdoWrapper and SimplePdo to use short nullables (e.g., ?string instead of string|null) and simplified array types for better type accuracy. [1] [2]

Static analysis configuration:

  • Modified phpstan.dist.neon to remove certain paths (vendor, flight/util/ReturnTypeWillChange.php) from the excluded paths, potentially increasing code coverage for static analysis.

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.

1 participant