Skip to content

Conversation

@samsonasik
Copy link
Member

@TomasVotruba let's improve UsefulArrayTagNodeAnalyzer to allow dummy mixed[] spacing aware array type node, which mostly won't be used as Templated Tag:

    /**
-    * @return mixed[]
+    * @return int[][]
     */

…rrayTypeNode array on UsefulArrayTagNodeAnalyzer
@samsonasik
Copy link
Member Author

All checks have passed 🎉 @TomasVotruba I think it is ready.

@samsonasik samsonasik marked this pull request as draft October 1, 2025 16:23
@samsonasik
Copy link
Member Author

samsonasik commented Oct 1, 2025

Rectify seems cause PHPStan error.

It seems when public method changed based on external call, it cause PHPStan error as different passed data.

@samsonasik
Copy link
Member Author

samsonasik commented Oct 1, 2025

I reverted rectify 17215ca

Then skip public method with mixed[] array docblock on ClassMethodArrayDocblockParamFromLocalCallsRector 641bc81 as the caller can be anything that applicable for mixed[]

@samsonasik samsonasik marked this pull request as ready for review October 1, 2025 16:39
@samsonasik
Copy link
Member Author

Fixed 👍

@samsonasik
Copy link
Member Author

All checks have passed 🎉 @TomasVotruba I think it is ready.

@TomasVotruba
Copy link
Member

Very nice, even few improvements in our codebase 👍

@TomasVotruba TomasVotruba merged commit 66c3294 into main Oct 1, 2025
50 checks passed
@TomasVotruba TomasVotruba deleted the allow-override-dummy-mixed-array branch October 1, 2025 17:41
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