Skip to content

Improve intersection of ConstantArray and AccessoryIsList#5067

Merged
ondrejmirtes merged 10 commits intophpstan:2.1.xfrom
VincentLanglet:experimental/array-list
Mar 11, 2026
Merged

Improve intersection of ConstantArray and AccessoryIsList#5067
ondrejmirtes merged 10 commits intophpstan:2.1.xfrom
VincentLanglet:experimental/array-list

Conversation

@VincentLanglet
Copy link
Contributor

@VincentLanglet VincentLanglet commented Feb 27, 2026

Proof of the gain in the next PR #5069

@VincentLanglet VincentLanglet force-pushed the experimental/array-list branch from 93ff38c to 65797b9 Compare February 27, 2026 15:38
@VincentLanglet VincentLanglet marked this pull request as ready for review February 27, 2026 15:45
@phpstan-bot
Copy link
Collaborator

This pull request has been marked as ready for review.

@VincentLanglet
Copy link
Contributor Author

This solve phpstan/phpstan#11602 (comment) but not the whole issue.

@VincentLanglet VincentLanglet force-pushed the experimental/array-list branch from 4e14478 to 355d824 Compare February 27, 2026 23:37
Copy link
Contributor

@staabm staabm left a comment

Choose a reason for hiding this comment

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

I think this makes sense. Ondrej should have another look though.

@VincentLanglet
Copy link
Contributor Author

VincentLanglet commented Mar 1, 2026

I think this makes sense. Ondrej should have another look though.

I agree ; please take a look @ondrejmirtes

@ondrejmirtes ondrejmirtes merged commit 951d9e4 into phpstan:2.1.x Mar 11, 2026
636 of 647 checks passed
@ondrejmirtes
Copy link
Member

Thank you! This is how I'd approach it too 👍

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.

4 participants