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

Pylint crash while pre-commit #9240

Closed
shmir opened this issue Nov 16, 2023 · 3 comments
Closed

Pylint crash while pre-commit #9240

shmir opened this issue Nov 16, 2023 · 3 comments
Labels
Duplicate 🐫 Duplicate of an already existing issue python 3.12

Comments

@shmir
Copy link

shmir commented Nov 16, 2023

pylint-crash-2023-11-16-14-50-53.txt

This is the pre-commit config:

  • repo: local
    hooks:
    • id: pylint
      name: pylint
      entry: pylint
      language: system
      types: [python]
      args: [
      --max-line-length=127,
      '--disable=logging-fstring-interpolation,logging-not-lazy,unspecified-encoding,logging-format-interpolation',
      '--good-names=_,rc,ti,ip,l3',
      '--load-plugins=pylint_pytest'
      ]

Python version is 12.

@jacobtylerwalls
Copy link
Member

Can you provide the output of pylint --version? Python 3.12 is only supported with pylint 3.0 and astroid 3.0.

@jacobtylerwalls
Copy link
Member

Duplicate of #9091

@jacobtylerwalls jacobtylerwalls marked this as a duplicate of #9091 Nov 16, 2023
@jacobtylerwalls jacobtylerwalls closed this as not planned Won't fix, can't repro, duplicate, stale Nov 16, 2023
@jacobtylerwalls jacobtylerwalls added the Duplicate 🐫 Duplicate of an already existing issue label Nov 16, 2023
@shmir
Copy link
Author

shmir commented Nov 16, 2023

After removing pylint-pytest and upgrading pylint as foll0wing:

(venv12) ~/PycharmProjects/multimedia (yoram-production) $ pip show pylint
Name: pylint
Version: 3.0.2
Summary: python code static checker
Home-page:
Author:
Author-email: Python Code Quality Authority code-quality@python.org
License: GPL-2.0-or-later
Location: /home/yorams/PycharmProjects/multimedia/venv12/lib/python3.12/site-packages
Requires: astroid, dill, dill, isort, mccabe, platformdirs, tomlkit
Required-by:
(venv12) ~/PycharmProjects/multimedia (yoram-production) $ pip show astroid
Name: astroid
Version: 3.0.1
Summary: An abstract syntax tree for Python with inference support.
Home-page:
Author:
Author-email:
License: LGPL-2.1-or-later
Location: /home/yorams/PycharmProjects/multimedia/venv12/lib/python3.12/site-packages
Requires:
Required-by: pylint
(venv12) ~/PycharmProjects/multimedia (yoram-production) $

It seems to be working.
Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Duplicate 🐫 Duplicate of an already existing issue python 3.12
Projects
None yet
Development

No branches or pull requests

2 participants