Skip to content

Disable treatPhpDocTypesAsCertain phpstan option and clean up baseline - #12631

Merged
greg0ire merged 1 commit into
doctrine:3.7.xfrom
whataboutpereira:treat-php-doc-types-as-certain
Sep 23, 2026
Merged

greg0ire merged 1 commit into
doctrine:3.7.xfrom
whataboutpereira:treat-php-doc-types-as-certain

Conversation

@whataboutpereira

Copy link
Copy Markdown
Contributor

Suggested in #12626 (comment)

This will allow for more defensive checks such as using narrower phpstan parameters that are also checked during runtime, without having to add phpstan ignores.

@whataboutpereira
whataboutpereira force-pushed the treat-php-doc-types-as-certain branch from 30510c0 to 1c832fe Compare September 22, 2026 15:00

@greg0ire greg0ire left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Doesn't look too bad indeed, I was expecting lots of additional issues because we trusted phpdoc, but maybe this is asymmetrical: it no longer blames us for triple checking, but doesn't blame us if we don't?

Comment thread src/Query/Expr/Base.php
…ine and @PHPStan-Ignore

This will allow for more defensive checks such as using narrower phpstan parameters that are also checked during runtime, without having to add phpstan ignores.

Suggested in #12626 (comment)

@SenseException SenseException left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@greg0ire greg0ire added this to the 3.7.2 milestone Sep 23, 2026
@greg0ire
greg0ire merged commit 6b47a02 into doctrine:3.7.x Sep 23, 2026
131 checks passed
@greg0ire

Copy link
Copy Markdown
Member

Thanks @whataboutpereira !

@whataboutpereira
whataboutpereira deleted the treat-php-doc-types-as-certain branch September 23, 2026 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants