Skip to content
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

update extra_pylint_args setting for list input #64

Open
bytesofmyself opened this issue Jan 17, 2017 · 0 comments
Open

update extra_pylint_args setting for list input #64

bytesofmyself opened this issue Jan 17, 2017 · 0 comments

Comments

@bytesofmyself
Copy link

This was probably the original intention of this setting but currently it only allows one argument to be passed as a string. More than one argument causes both arguments to fail unless the first argument ends in a comma and even then any additional argument fails.

Works:
"pylint_extra": "--ignored-classes=MsgProto"

Both args fail:
"pylint_extra": "--ignored-classes=MsgProto --extension-pkg-whitelist=lxml,cassandra"

First arg works:
"pylint_extra": "--ignored-classes=MsgProto, --extension-pkg-whitelist=lxml,cassandra"

This setting should instead a list of strings instead of a string
"pylint_extra": ["--ignored-classes=MsgProto", "--extension-pkg-whitelist=lxml,cassandra"]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant