Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
merged branch lyrixx/feat-auto-suggest (PR #3325)
Commits ------- e5edf5a [Console] Fixed CS 8abf506 [Console] Added abbreviation into search for bad command / namespace c6203bc [Console] Added namespace suggest on bad namespace name 117359a [Console] fixed CS according to PR comment dd0d97e [Console] Added suggest on bad command name Discussion ---------- [Console] Added suggest on bad command name Bug fix: no Feature addition: yes Backwards compatibility break: no Symfony2 tests pass: yes Fixes the following tickets: - Todo: namespace ? Added something like in `git` : if user type a wrong command and if a close alternative exists, Command compenent will display a list of similar command(s). Note : It does not work with namespace. If this PR will be merged, I could work on namespace. see : https://github.com/fabpot/Twig/blob/master/lib/Twig/Environment.php#L1003 --------------------------------------------------------------------------- by fabpot at 2012-02-11T18:54:49Z I think we need it to also work on namespace before merging. Is it possible? --------------------------------------------------------------------------- by henrikbjorn at 2012-02-11T19:01:06Z could maybe use similar_text ? --------------------------------------------------------------------------- by lyrixx at 2012-02-11T19:01:55Z Yes. I will work on it asap --------------------------------------------------------------------------- by lyrixx at 2012-02-11T20:06:43Z I added code for namespace @henrikbjorn I did the same logic as in twig. --------------------------------------------------------------------------- by lyrixx at 2012-02-11T20:27:48Z Note : Travis tests failed : http://travis-ci.org/#!/lyrixx/symfony/builds/663216 ```before_script: Execution of 'php vendors.php' took longer than 600 seconds and was terminated. Consider rewriting your stuff in AssemblyScript, we've heard it handles Web Scale™``` But tests are OK on my laptop --------------------------------------------------------------------------- by stof at 2012-02-11T20:41:15Z Well, it may be due to github issues during the setup of the vendors. There is some issues regularly because of the DDoS attack. --------------------------------------------------------------------------- by lyrixx at 2012-02-11T20:58:07Z Yes, i guessed it :-) that's why i notice it work on my laptop --------------------------------------------------------------------------- by fabpot at 2012-02-11T23:11:08Z This code won't work if you use abbreviations instead of the full namespace or command name. --------------------------------------------------------------------------- by lyrixx at 2012-02-12T23:30:04Z I added code to manage abbreviations. But I'm not sure what you are expecting. Can you try it and give me some feedback ? P.S. : Travis failed again, but tests pass on my laptop.
- Loading branch information
Showing
2 changed files
with
165 additions
and
8 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters