Too early to include it on Package Control ?

[bizoo]

Hi, I just take a look at the new candidates for Package Control and checked your package.

No offense, but didn't you think this is a work in progress package not ready to be released to public ?

Just a few things I noticed:

• ignoreDirs must not be a constant, use settings().get('ignored_packages').
• json with comments raise an exception.
• Why didn't parse Default and add sysCommand ? (because of exception on json parsing when comments are present ?)
• results are not sorted ?

[welefen]

• ignoreDirs must not be a constant, use settings().get('ignored_packages'). -- fixed
• json with comments raise an exception. -- add try
• Why didn't parse Default and add sysCommand ? -- Default has too many keymap, so only show some very usefull keymap
• results are not sorted ? -- sorted

[bizoo]

Nice, thanks.