Add DocBlock so IDEs do not complain

edvinaskrucas / notification

Notification package for Laravel

MIT License

525 stars 99 forks source link

Add DocBlock so IDEs do not complain #74

Closed Kyslik closed 7 years ago

Kyslik commented 8 years ago

I added all methods that I found inside readme file.

If anyone is interested in topic http://stackoverflow.com/a/15634488/1564365

PHPStorm IDE tested

Tryouts

If anyone wants to use this and this PR is not merged yet use this:

composer.json

"repositories": [
  {
    "type": "vcs",
    "url": "https://github.com/Kyslik/notification"
  }
],
"require": {
  "edvinaskrucas/notification": "dev-docblock",
},

Kurre commented 8 years ago

Thanks for your effort!

I asked about this on https://github.com/edvinaskrucas/notification/issues/36 and it's nice to see someone (you) had time to pull this together :+1:

I'll also test this asap, but looks ok to me.

Hopefully this will get merged soon :)

Kyslik commented 8 years ago

If its not merged yet you can use my fork. If you find any method that is missing make PR.

And no problem && enjoy autocompletion :).