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

[2.12.1 Regression] 'LinterStats' object has no attribute 'get' #5426

Closed
dzhengfy opened this issue Nov 29, 2021 · 4 comments
Closed

[2.12.1 Regression] 'LinterStats' object has no attribute 'get' #5426

dzhengfy opened this issue Nov 29, 2021 · 4 comments
Labels
Bug 🪲 Downstream Bug 🪲 The problem happens in a lib depending on pylint, not pylint
Milestone

Comments

@dzhengfy
Copy link

Bug description

Refer link: https://github.com/autotest/tp-libvirt/runs/4330039462?check_suite_focus=true
Tp-libvirt project is using pylint for python checking. With pylint 2.11.0, everything is PASS. After upgrading to pylint 2.12, every PR checking is failed with below error message.

Run inspekt checkall --disable-style E501,E265,W601,W605,E402,E722,E741 --no-license-check
PEP8 disabled: E501,E265,W601,W605,E402,E722,E741
Pylint disabled: W,R,C,E1002,E1101,E1103,E1120,F0401,I0011
Pylint enabled : W0611
License check: disabled
'LinterStats' object has no attribute 'get'
Error: Process completed with exit code 1.

Configuration

No response

Command used

inspekt checkall --disable-style E501,E265,W601,W605,E402,E722,E741 --no-license-check

Pylint output

'LinterStats' object has no attribute 'get'

Expected behavior

Checking is PASSed.

Pylint version

pylint-2.12.1

OS / Environment

No response

Additional dependencies

No response

@dzhengfy dzhengfy added Bug 🪲 Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling labels Nov 29, 2021
@DanielNoord
Copy link
Collaborator

This is not actually a bug in pylint, but a bug in inspektor which is using elements of pylint's API that we did not consider public.
A PR has already been opened with inspektor to fix this, see avocado-framework/inspektor#41.

I'll respond to that PR with a better fix to solve the issue!

@Pierre-Sassoulas Pierre-Sassoulas added Downstream Bug 🪲 The problem happens in a lib depending on pylint, not pylint and removed Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling labels Nov 29, 2021
@dzhengfy
Copy link
Author

@DanielNoord Thank you so much for your help.

@Pierre-Sassoulas Pierre-Sassoulas added this to the 2.13.0 milestone Nov 30, 2021
@Pierre-Sassoulas
Copy link
Member

The MR in avocado-framework/inspektor#41 seems well on it's way with the precious help from @DanielNoord I added this issue to 2.13, so we close it at some point.

@DanielNoord
Copy link
Collaborator

Closing as the downstream issue has been fixed! Thanks for the report @dzhengfy!

@Pierre-Sassoulas Pierre-Sassoulas modified the milestones: 2.13.0, 2.12.2 Dec 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 🪲 Downstream Bug 🪲 The problem happens in a lib depending on pylint, not pylint
Projects
None yet
Development

No branches or pull requests

3 participants