search

padawan-php / padawan.php

php intelligent code completion http server
http://mkusher.name/padawan/
MIT License
253 stars 30 forks source link

Discoverability #50

Closed greg0ire closed 8 years ago

greg0ire commented 8 years ago

On my setup, typing just bin/padawan or bin/padawan made-up argument does nothing. I would expect that to give a list of possible commands / the help for padawan.

greg0ire commented 8 years ago

Looking at the code, I think having an ErrorCommand makes no sense, and an exception should be thrown instead. Also, how about symfony's Console component? It should make all this easy.

greg0ire commented 8 years ago

Apparently, the console component is already an implicit dependency of the project : 

  - Installing symfony/console (v2.8.8)
    Downloading: 100%
mkusher commented 8 years ago

@greg0ire version in master does it

greg0ire commented 8 years ago

Oh… ooooh. I feel stupid right now. Closing, sorry.

mkusher commented 8 years ago

@greg0ire it's ok, cause there is still no stable release with this fix :(

greg0ire commented 8 years ago

May I ask, why don't you release more often ? The last release is almost one year old :confused:

mkusher commented 8 years ago

@greg0ire planned to much features to add :(

greg0ire commented 8 years ago

Feels very familiar :) Do you have a moment to talk about our lords and saviours time-based release? I think switching to that kind of release would spare you some stupid issues like this one, and to get earlier feedback on new things, while making upgrade less painful. The only drawback I can see is that it will force you to have higher version numbers quicker (think firefox 47), but I don't think anyone really cares about that, so please consider it.

mkusher commented 8 years ago

it's a good idea, I'll need to more accurately manage my time, so I'll have more time to spend on padawan