Skip to content

Conversation

@staabm
Copy link
Contributor

@staabm staabm commented Dec 9, 2025

fixes PHP 7.4 build errors

@staabm staabm marked this pull request as ready for review December 9, 2025 07:30
@phpstan-bot
Copy link
Collaborator

This pull request has been marked as ready for review.

Comment on lines +117 to +130
-
message: '#^Parameter \#3 of function sprintf is expected to be string by placeholder \#2 \("%%s"\), string\|false given\.$#'
count: 1
path: ../src/Cache/FileCacheStorage.php

-
message: '#^Parameter \#3 of function sprintf is expected to be string by placeholder \#2 \("%%s"\), string\|false given\.$#'
count: 1
path: ../src/Rules/Cast/InvalidCastRule.php

-
message: '#^Parameter \#2 of function sprintf is expected to be string by placeholder \#1 \("%%s"\), string\|false given\.$#'
count: 1
path: ../src/Rules/ClassForbiddenNameCheck.php
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tried to make the type inference more useful, but in the end everything I did which fixed php 7.x errors, created new php8 errors.

therefore added php7.x only ignores and called it a day

@ondrejmirtes ondrejmirtes merged commit 1fb2fdb into phpstan:2.1.x Dec 9, 2025
627 of 635 checks passed
@ondrejmirtes
Copy link
Member

Thank you!

@staabm staabm deleted the fix7 branch December 9, 2025 08:46
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