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

Fix phpdoc types #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix phpdoc types #6

wants to merge 1 commit into from

Conversation

KapitanOczywisty
Copy link
Owner

Quick fix for phpdoc types, there is more changes needed to implement more complex types e.g. generics

Fixed problems:

  • Consuming multiple lines
    image
    image
  • Namespace not highlighted
    image

@BladeMF
Copy link

BladeMF commented Oct 30, 2023

Do you need help with this? I tried helping the original repository, but I am being ignored there. I think what is missing are these cases:
image
I'd love to put all the phpstan and psalm tags as well.

@KapitanOczywisty
Copy link
Owner Author

@BladeMF This PR is just dirty fix and should be completed, just I didn't get around to merge it. It'd be nice if you could review it. Feel free to add phpstan and psalm keywords.

@BladeMF
Copy link

BladeMF commented Oct 31, 2023

I'd rather start doing small PRs for each feature one at a time, including yours. I can create a task list and tick them off.

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.

2 participants