-
Notifications
You must be signed in to change notification settings - Fork 126
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TypeError: sequence item 0: expected string, NoneType found #13
Comments
Hi there. Which version of Ubuntu as I test this in 11.10 at the moment? |
This seems to be related to when Sublime cannot find the settings file, so setting a default from within the plugin as well - #13
I think this is happening when/if Sublime cannot find the settings file. So I've committed a fix which is on the master branch, please check to see if this works for you. |
works now! Didn't know package control automatically updates plugins on start :) By the way, I'm using ubuntu 12.04. |
I'll start to add 12.04 to my list of OS's to check. Package Control is fantastic ;-) |
It seems Pref.phpcs_executable_path equates to None in ubuntu @ "phpcs.py" 362L thus args first item is not phpcs
I temporarily changed it to
if (Pref.phpcs_executable_path != "") and Pref.phpcs_executable_path != None:
I have very basic python knowledge.
The text was updated successfully, but these errors were encountered: