-
Notifications
You must be signed in to change notification settings - Fork 1
Symfony autocomplete #217
base: trunk
Are you sure you want to change the base?
Symfony autocomplete #217
Conversation
|
@ahegyes how do you feel about this. When the user writes a command and hits
-- |
I'm in favor of not doing that simply because I think it kinda clutters the terminal and I haven't seen it done anywhere else. But I can see some value in it. However, is the example auto-generated or will that be another thing to remember to update if the options change? 🤔 |
|
@ahegyes Yeah that's ok, it was an idea I had and wanted to hear opinions, but I wasn't 100% sold on it either. I haven't found a way to make it auto-generated so that's a con, it would need to be manually updated 👎 |
|
@ahegyes some recent updates:
Would you mind testing when you have a chance? |
34f1392 to
be3a6a0
Compare
Also added int return type and changed.
|
@fmfernandes since you're already familiar with this problem and its solution, would you mind taking a look at commit eae6cb4 and let me know if you see anything suspicious or have any suggestion or question? |
fmfernandes
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR looks good. Left a couple comments regarding some return types I'd change to match the command's output.
|
Thanks for those observations @fmfernandes, I've updated both cases here: c5978c9 |

WIP