Skip to content

Support PHPStan Level 7#1652

Merged
KajitoriAdmin merged 7 commits into
exceedone:masterfrom
oreno4649:feat/phpstan-lv7
May 13, 2026
Merged

Support PHPStan Level 7#1652
KajitoriAdmin merged 7 commits into
exceedone:masterfrom
oreno4649:feat/phpstan-lv7

Conversation

@oreno4649
Copy link
Copy Markdown
Collaborator

@oreno4649 oreno4649 commented Nov 28, 2025

PHP Stan Level 7 をサポートしました

@oreno4649 oreno4649 self-assigned this Nov 30, 2025
# Conflicts:
#	src/Services/Installer/DatabaseForm.php
The temporary "test: github actions" override pointed at oreno4649/docker-exment#fix/mysql-warning, which has since been merged and deleted, causing all CI checkouts to fail.
Revert the 47 src/ files that had behavioral code modifications (return type declarations, (string)trans casts, variable extractions/renames, null-safety guards, explicit return null, constructor reorder) back to upstream/master state. Add // @phpstan-ignore-next-line above each affected line so PHPStan Level 7 still passes without changing runtime behavior.
…nores

PHPStan flagged 69 // @phpstan-ignore-next-line comments as "No error to ignore" because the original code did not produce errors at level 7 once the runtime modifications were reverted. Remove those directives. For RouteOAuthServiceProvider, the genuine error sits on the Passport::tokensExpireIn() argument line, so move the ignore directives directly above those calls.
@KajitoriAdmin KajitoriAdmin merged commit 9113a27 into exceedone:master May 13, 2026
10 checks passed
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