usmanhalalit / pixie

Database query builder for PHP, framework agnostic, lightweight and expressive.
MIT License
672 stars 180 forks source link

Update docblock to include data types and better support IDEs #167

Closed TCB13 closed 6 years ago

TCB13 commented 7 years ago

Hello,

I've noticed that the docblock isn't very compatible with Phpstrom and 4k IDEs because it doesn't declare the datatype of the parameters.

Here is the declaration of table(): captura de ecra 2017-06-29 as 10 49 02

Since it lacks the datatype, Phpstorm complains right there... and then at my code:

captura de ecra 2017-06-29 as 10 49 14

It is reading the "Single" as the datatype because there should be there an extra field specifying the datatype.

Have a nice day.

Finesse commented 7 years ago

This comments are called PHPDoc. I join the issue.

TCB13 commented 7 years ago

@FinesseRus check #150 .

TCB13 commented 6 years ago

Discussion continues on #150 .