Skip to content

Conversation

@zonuexe
Copy link
Contributor

@zonuexe zonuexe commented Sep 25, 2024

This PR is still a work in progress, no tests or rules have been added yet.

  • Add tests
  • Add rules
  • Add function map

refs #3106, phpstan/phpstan#11100 and phpstan/phpstan#11710
closes phpstan/phpstan#11101

related issue phpstan/phpdoc-parser#253

@zonuexe zonuexe marked this pull request as draft September 25, 2024 17:46
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will be replaced by phpstan/phpdoc-parser#253

Copy link
Contributor Author

Choose a reason for hiding this comment

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

WIP. = false will be removed.

@ondrejmirtes
Copy link
Member

Just updated phpdoc-parser here so you can use that.

@zonuexe zonuexe force-pushed the feature/pure-unless-callable-is-impure branch 4 times, most recently from eddd22a to f207c72 Compare September 30, 2024 18:09
@zonuexe zonuexe force-pushed the feature/pure-unless-callable-is-impure branch from f207c72 to 8c756cc Compare November 3, 2024 17:57
@staabm
Copy link
Contributor

staabm commented Nov 18, 2024

@zonuexe any plans to move this forward?

@zonuexe
Copy link
Contributor Author

zonuexe commented Nov 19, 2024

@staabm Hi, I know this branch is important but it was blocked due to my motivation. I'm going to rebase it on 2.0.x and do the necessary work. If I miss it tonight, I won't be able to start the rest of the work until next week, so it would be quicker for you to take over then.

@staabm
Copy link
Contributor

staabm commented Nov 19, 2024

I don't want to put pressure on you. It was not clear to me whether you stopped working on it or not.

I am totally fine if this takes a bit more time - thank you

@zonuexe zonuexe force-pushed the feature/pure-unless-callable-is-impure branch from 8c756cc to 389c7f0 Compare November 19, 2024 16:58
@zonuexe
Copy link
Contributor Author

zonuexe commented Nov 19, 2024

No problem, this is a very interesting subject for me too, so I'm happy to work on it because I realized that there are other people who are interested in it. More importantly, I'm always helped by your hard work.

@zonuexe zonuexe changed the base branch from 1.12.x to 2.0.x November 19, 2024 17:11
@zonuexe zonuexe force-pushed the feature/pure-unless-callable-is-impure branch 4 times, most recently from 2146ae0 to 1cc1fad Compare November 19, 2024 18:32
@VincentLanglet
Copy link
Contributor

Friendly ping @zonuexe, still interested in this PR ? Do you have time to finish it ?
Do you look for help ? What is missing ?

@zonuexe
Copy link
Contributor Author

zonuexe commented Jul 20, 2025

@VincentLanglet
Thank you for the suggestion, sorry for making you wait so long.
I've been working on PHPStan 2.1.18 a few times over the last few weeks, and just finished a PHPStan workshop at a local PHP conference, so I'll get back to that tomorrow.

@zonuexe zonuexe force-pushed the feature/pure-unless-callable-is-impure branch from adb02ca to abc8cd8 Compare August 1, 2025 12:45
@zonuexe zonuexe changed the base branch from 2.0.x to 2.1.x August 1, 2025 12:46
@zonuexe zonuexe force-pushed the feature/pure-unless-callable-is-impure branch from abc8cd8 to c4999be Compare August 1, 2025 13:09
@VincentLanglet
Copy link
Contributor

Friendly ping @zonuexe

I don't want to put pressure on you. It's just not clear to me whether you stopped working on it or not.
What is missing ?

It will unblock the phpstan/phpdoc-parser#259 pr :)

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

Labels

None yet

4 participants