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

Some CLI disables are ignored with multiple jobs #2674

Closed
NotSqrt opened this issue Jan 3, 2019 · 3 comments
Closed

Some CLI disables are ignored with multiple jobs #2674

NotSqrt opened this issue Jan 3, 2019 · 3 comments

Comments

@NotSqrt
Copy link

NotSqrt commented Jan 3, 2019

Steps to reproduce

  1. echo "import os" >> example.py
  2. pylint --py3k --disable=no-absolute-import -j 1 example.py
  3. pylint --py3k --disable=no-absolute-import -j 2 example.py

Current behavior

No errors with 1 job.

Output with 2 jobs:

************* Module example
example.py:9:0: W1618: import missing `from __future__ import absolute_import` (no-absolute-import)

Expected behavior

No difference when changing number of jobs.

Probably related to other issues on differences when changing number of jobs (#374 for example).

pylint --version output

pylint 2.2.2
astroid 2.1.0
Python 3.6.7 (default, Oct 21 2018, 04:56:05) 
[GCC 5.4.0 20160609]

Thanks !

@PCManticore
Copy link
Contributor

Thanks, I can reproduce the bug. There must be some configuration issue between the main process and the child linter processes when using multiple jobs.

@doublethefish
Copy link
Contributor

doublethefish commented Mar 30, 2020

This looks fixed in latest master.HEAD (fails in 2.4.4)

I've put it under test:
doublethefish@c7288b7

Travis results:
https://travis-ci.org/github/doublethefish/pylint/builds/668832863

@Pierre-Sassoulas
Copy link
Member

This was done in #4173 and never closed, then we removed the python3 porting mode in 2.11.

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

No branches or pull requests

4 participants