Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow @phpstan-use and @use tags to be declared into class phpdoc #2638

Open
wants to merge 1 commit into
base: 1.10.x
Choose a base branch
from

Conversation

baptistepillot
Copy link

This implements the feature request here: phpstan/phpstan#7486.
I think this does the job. Hope this will help.

@baptistepillot
Copy link
Author

php 7.4's phpstan-doctrine fails because of Allowed memory size of 805306368 bytes exhausted (tried to allocate 20480 bytes).
The same as for a number of recent merge requests. phpstan eats a big bucket of memory for breakfast, seems that php 7.4 + Doctrine breaks the CI chain limited memory down.

@baptistepillot
Copy link
Author

Hello there. What's up ? Do you envisage a merge date for this pull request ?
I am ready to rebase this before merging when you wish.

@mxr576
Copy link

mxr576 commented May 2, 2024

The current default branch is https://github.com/phpstan/phpstan-src/tree/1.11.x so the target should be changed for a start.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants