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

Allow linting directories without __init__.py #3569

Merged
merged 1 commit into from
May 2, 2020

Conversation

PCManticore
Copy link
Contributor

Description

This was a regression in 2.5 caused by a6d7ffc.

Close #3528

Type of Changes

Type
βœ“ πŸ› Bug fix

This was a regressin in 2.5.
Close #3528
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 90.403% when pulling ab891ae on allow-linting-directories-without-init into 54ce3ba on 2.5.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 90.403% when pulling ab891ae on allow-linting-directories-without-init into 54ce3ba on 2.5.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 90.403% when pulling ab891ae on allow-linting-directories-without-init into 54ce3ba on 2.5.

@coveralls
Copy link

coveralls commented May 2, 2020

Coverage Status

Coverage increased (+0.02%) to 90.447% when pulling ab891ae on allow-linting-directories-without-init into 54ce3ba on 2.5.

Copy link
Member

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pip3 install .;python3 pylint/__main__.py module
Processing /home/pierre/PycharmProjects/pylint
Collecting astroid<=2.5,>=2.4.0 (from pylint==2.5.0)
  Using cached https://files.pythonhosted.org/packages/7c/f6/880bade8329525fd9aa9051891b4b8fe2e52c7aab7412443f8743028a590/astroid-2.4.0-py3-none-any.whl
Requirement already satisfied: isort<5,>=4.2.5 in ./venv/lib/python3.6/site-packages (from pylint==2.5.0)
Requirement already satisfied: mccabe<0.7,>=0.6 in ./venv/lib/python3.6/site-packages (from pylint==2.5.0)
Requirement already satisfied: toml>=0.7.1 in ./venv/lib/python3.6/site-packages (from pylint==2.5.0)
Requirement already satisfied: typed-ast<1.5,>=1.4.0; implementation_name == "cpython" and python_version < "3.8" in ./venv/lib/python3.6/site-packages (from astroid<=2.5,>=2.4.0->pylint==2.5.0)
Requirement already satisfied: lazy-object-proxy==1.4.* in ./venv/lib/python3.6/site-packages (from astroid<=2.5,>=2.4.0->pylint==2.5.0)
Requirement already satisfied: wrapt~=1.11 in ./venv/lib/python3.6/site-packages (from astroid<=2.5,>=2.4.0->pylint==2.5.0)
Requirement already satisfied: six~=1.12 in ./venv/lib/python3.6/site-packages (from astroid<=2.5,>=2.4.0->pylint==2.5.0)
Installing collected packages: astroid, pylint
  Found existing installation: astroid 2.3.3
    Uninstalling astroid-2.3.3:
      Successfully uninstalled astroid-2.3.3
  Found existing installation: pylint 2.4.4
    Uninstalling pylint-2.4.4:
      Successfully uninstalled pylint-2.4.4
  Running setup.py install for pylint ... done
Successfully installed astroid-2.4.0 pylint-2.5.0

--------------------------------------------------------------------
Your code has been rated at 10.00/10 (previous run: 10.00/10, +0.00)

And damn, it was simpler to fix than I thought :D Could the base problem with "pylint ." not working be fixed in a similar "simple" manner?

@PCManticore
Copy link
Contributor Author

Not sure about pylint ., I believe we just need a way to find all files in the given location to solve that issue.

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

Successfully merging this pull request may close these issues.

3 participants