Skip to content

Conversation

@staabm
Copy link
Contributor

@staabm staabm commented Sep 18, 2024

required for phpstan/phpstan#11100

@staabm staabm marked this pull request as ready for review September 18, 2024 14:10
@ondrejmirtes
Copy link
Member

Pinging some people in the community about an opinion first, stay tuned :)

@ondrejmirtes
Copy link
Member

Decided to go for it. Other suggestions similar to how Hack handles things are more general, but very complicated to do right now.

@ondrejmirtes ondrejmirtes merged commit 6ca22b1 into phpstan:1.23.x Sep 26, 2024
56 of 57 checks passed
@ondrejmirtes
Copy link
Member

Thank you!

@staabm staabm deleted the pure-cb branch September 26, 2024 07:55
@hrach
Copy link

hrach commented Sep 26, 2024

What a missed opportunity to name it @pure-iff-callable-is-pure

@ondrejmirtes
Copy link
Member

iff is something math nerds use 😂

@hrach
Copy link

hrach commented Sep 26, 2024

Maybe an alias? 😇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants