search

KnpLabs / ConsoleServiceProvider

A Console service provider for Silex
MIT License
107 stars 37 forks source link

Command - added docblock for getApplication() #26

Closed ovr closed 8 years ago

ovr commented 8 years ago

Hey!

image

It's needed to navigate

Thanks

ghost commented 8 years ago

I believe it should be more like this?

screen shot 2016-05-31 at 12 34 42 pm
ovr commented 8 years ago

In my commit I declare that getApplication will return Knp Console Application And after it IDE can suggest getSilexApplication() and etc moments from Knp Console App

In you case you declare method that it exists but in what class?

I think my variant is more correct

Thanks