Skip to content

Parallel jobs don't work with custom plugins #456

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

Closed
pylint-bot opened this issue Jan 26, 2015 · 9 comments
Closed

Parallel jobs don't work with custom plugins #456

pylint-bot opened this issue Jan 26, 2015 · 9 comments
Labels

Comments

@pylint-bot
Copy link

Originally reported by: Pedro Algarvio (BitBucket: s0undt3ch, GitHub: @s0undt3ch?)


I've created some PyLint plugins which define some options.

When running PyLint with jobs set to more than 1, it fails badly:

Process ChildLinter-2:
Traceback (most recent call last):
  File "/home/vampas/.dotfiles/.ext/pyenv/versions/2.7.8/lib/python2.7/multiprocessing/process.py", line 258, in _bootstrap
    self.run()
  File "/home/vampas/.dotfiles/.ext/pyenv/versions/SaltPy27/lib/python2.7/site-packages/pylint/lint.py", line 196, in run
    result = self._run_linter(file_or_module[0])
  File "/home/vampas/.dotfiles/.ext/pyenv/versions/SaltPy27/lib/python2.7/site-packages/pylint/lint.py", line 212, in _run_linter
    linter.load_configuration(**self._config)
  File "/home/vampas/.dotfiles/.ext/pyenv/versions/SaltPy27/lib/python2.7/site-packages/logilab/common/configuration.py", line 682, in load_configuration
    provider = self._all_options[opt]
KeyError: 'un-indexed-curly-braces-always-error'

My Plugin's option definition can be found here.


@pylint-bot
Copy link
Author

Original comment by Pedro Algarvio (BitBucket: s0undt3ch, GitHub: @s0undt3ch?):


❯ pylint --version                                                                                                                                    features/pylint-py3-modernize 0ce7760 ✗
pylint 1.4.1, 
astroid 1.3.4, common 0.63.2
Python 2.7.8 (default, Oct 27 2014, 04:23:15) 
[GCC 4.9.1 20140903 (prerelease)]

@pylint-bot
Copy link
Author

Original comment by Claudiu Popa (BitBucket: PCManticore, GitHub: @PCManticore):


--jobs can be used with --load-plugins now. Closes issue #456.

@pylint-bot
Copy link
Author

Original comment by Claudiu Popa (BitBucket: PCManticore, GitHub: @PCManticore):


Should be fixed now.

@pylint-bot
Copy link
Author

Original comment by Pedro Algarvio (BitBucket: s0undt3ch, GitHub: @s0undt3ch?):


Still not fixed:

#!python

❯ pylint --rcfile=.testing.pylintrc --disable=W1307 salt/                                                                                                                   develop 054bd9b ✗
Process ChildLinter-2:
Traceback (most recent call last):
  File "/home/vampas/.dotfiles/.ext/pyenv/versions/2.7.8/lib/python2.7/multiprocessing/process.py", line 258, in _bootstrap
    self.run()
  File "/home/vampas/.dotfiles/.ext/pyenv/versions/SaltPy27/lib/python2.7/site-packages/pylint/lint.py", line 220, in run
    result = self._run_linter(file_or_module[0])
  File "/home/vampas/.dotfiles/.ext/pyenv/versions/SaltPy27/lib/python2.7/site-packages/pylint/lint.py", line 237, in _run_linter
    linter.load_configuration(**self._config)
  File "/home/vampas/.dotfiles/.ext/pyenv/versions/SaltPy27/lib/python2.7/site-packages/logilab/common/configuration.py", line 682, in load_configuration
    provider = self._all_options[opt]
KeyError: 'un-indexed-curly-braces-always-erro

@pylint-bot
Copy link
Author

Original comment by Pedro Algarvio (BitBucket: s0undt3ch, GitHub: @s0undt3ch?):


My current PyLint version

❯ pylint --version                                                                                                                                                          develop 054bd9b ✗
pylint 1.4.4, 
astroid 1.3.8, common 1.0.2
Python 2.7.8 (default, Oct 27 2014, 04:23:15) 
[GCC 4.9.1 20140903 (prerelease)]

@pylint-bot
Copy link
Author

Original comment by Claudiu Popa (BitBucket: PCManticore, GitHub: @PCManticore):


Could you try with pylint 1.5.0 (the repo version)?

@pylint-bot
Copy link
Author

Original comment by Pedro Algarvio (BitBucket: s0undt3ch, GitHub: @s0undt3ch?):


I'm guessing I also need the repo astroid version...

@pylint-bot
Copy link
Author

Original comment by Claudiu Popa (BitBucket: PCManticore, GitHub: @PCManticore):


Wow, that's a big wall of text!

I'm hoping two-three weeks, depends as soon as I finish Python 3.5 support.

@pylint-bot
Copy link
Author

Original comment by Pedro Algarvio (BitBucket: s0undt3ch, GitHub: @s0undt3ch?):


On 10-09-2015 15:38, Claudiu Popa wrote:

I'm guessing there are a lot of options being deprecated, hence the wall
of text...

Superb, Thanks!

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

No branches or pull requests

1 participant