Skip to content

Constant wildcard with Class::*|null or list<Class::*> #45

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.

Sign up for GitHub

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

Jump to bottom
Closed
jderusse opened this issue May 4, 2020 · 5 comments
Closed

Constant wildcard with Class::*|null or list<Class::*> #45

jderusse opened this issue May 4, 2020 · 5 comments

Comments

@jderusse
Copy link

jderusse commented May 4, 2020

Thanks for this new feature, but it fail to parse the constraint whenit's combined

list<QueueAttributeName::*>
ParameterTier::*|null
@jderusse jderusse changed the title Constant wildcard with CONST::*|null| or <CONS::*> Constant wildcard with Class::*|null or list<Class::*> May 4, 2020
jderusse added a commit to jderusse/phpdoc-parser that referenced this issue May 4, 2020
@jderusse
Exclude >, |, , from TOKEN_OTHER
f4a91af 
fixes phpstan#45
@ondrejmirtes
Copy link
Member

Fixed, thank you! It will take a few days before the next PHPStan gets released with this parser version: d840b69

@jderusse
Copy link
Author

jderusse commented May 4, 2020

Thanks a lot for the patch!

@jderusse jderusse mentioned this issue May 4, 2020
Merged
@ondrejmirtes
Copy link
Member

Released in PHPStan 0.12.23 :)

@jderusse
Copy link
Author

jderusse commented May 5, 2020

thanks

@github-actions
Copy link

github-actions bot commented May 1, 2021

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ondrejmirtes @jderusse