Converting to poetry for packaging instead of legacy setup.py method

Author: Carl Crowder

MANIFEST.in

@@ -43,9 +43,7 @@ def augment_visit(linter: PyLinter, checker_method, augmentation):
     try:
         checker = get_checker(linter, checker_method.__self__.__class__)
     except AttributeError:
-        checker = get_checker(
-            linter, get_class(checker_method.__module__, checker_method.__qualname__)
-        )
+        checker = get_checker(linter, get_class(checker_method.__module__, checker_method.__qualname__))
 
     old_method = getattr(checker, checker_method.__name__)
     setattr(checker, checker_method.__name__, AugmentFunc(old_method, augmentation))
@@ -101,9 +99,7 @@ def suppress_message(linter: PyLinter, checker_method, message_id_or_symbol, tes
     returns True. It is useful to prevent one particular message from being raised
     in one particular case, while leaving the rest of the messages intact.
     """
-    augment_visit(
-        linter, checker_method, DoSuppress(linter, message_id_or_symbol, test_func)
-    )
+    augment_visit(linter, checker_method, DoSuppress(linter, message_id_or_symbol, test_func))
 
 
 class DoSuppress:
@@ -161,8 +157,5 @@ def get_message_definitions(self, message_id_or_symbol):
         elif hasattr(msgs_store, "get_message_definitions"):
             return msgs_store.get_message_definitions(message_id_or_symbol)
         else:
-            msg = (
-                "pylint.utils.MessagesStore does not have a "
-                "get_message_definition(s) method"
-            )
+            msg = "pylint.utils.MessagesStore does not have a " "get_message_definition(s) method"
             raise ValueError(msg)

poetry.lock

@@ -0,0 +1,50 @@
+[tool.poetry]
+name = "pylint-plugin-utils"
+version = "0.8"
+readme = "README.md"
+description = "Utilities and helpers for writing Pylint plugins"
+repository = "https://github.com/PyCQA/pylint-plugin-utils"
+authors = ["Carl Crowder <[email protected]>"]
+license="GPLv2"
+keywords=["pylint","plugin","helpers"]
+packages = [
+  {include = "pylint_plugin_utils/"}
+]
+include = [
+  "LICENSE",
+  "*.rst",
+  "*.md"
+]
+
+[tool.poetry.dependencies]
+python = ">=3.7,<4.0"
+pylint = ">=1.7"
+
+[tool.poetry.group.dev.dependencies]
+pytest = "~6.2"
+pre-commit = "~2.16"
+pytest-cov = "~3.0"
+coveralls = "~3.3"
+tox = "^4.5.1"
+
+[build-system]
+requires = ["poetry-core>=1.0.0"]
+build-backend = "poetry.core.masonry.api"
+
+[tool.pytest.ini_options]
+python_files = ["tests.py","test_*.py","*_tests.py","tests/*.py"]
+
+[tool.black]
+line-length = 120
+exclude = '''
+/(
+    tests/
+)/
+'''
+
+[tool.isort]
+multi_line_output = 3
+include_trailing_comma = true
+force_grid_wrap = 0
+use_parentheses = true
+line_length = 120

poke-pre-commit-ci

@@ -17,9 +17,15 @@ def test_linter_should_be_pickleable(linter):
     # Setup
     linter.register_checker(TypeChecker())
     augment_visit(linter, TypeChecker.visit_attribute, fake_augmentation_func)
-    suppress_message(
-        linter, TypeChecker.visit_attribute, "no-member", fake_suppress_func
-    )
+    suppress_message(linter, TypeChecker.visit_attribute, "no-member", fake_suppress_func)
 
     # Act and Assert
-    pickle.dumps(linter)
+    try:
+        pickle.dumps(linter)
+    except TypeError:
+        # this can happen for newer versions of pylint (>2.12) which now use dill for pickling
+        import dill
+
+        # this import will fail for earlier versions which do not have dill as a dependency,
+        # in which case this is an old version which also does not pickle
+        dill.dumps(linter)

pylint_plugin_utils/__init__.py

@@ -1,32 +1,30 @@
 [tox]
 minversion = 2.4
-envlist = formatting, py36, py37, py38, py39, py310, pypy
+envlist = formatting, py36, py37, py38, py39, py310, py311, pypy
 skip_missing_interpreters = true
 requires = pip >=21.3.1
 
 [testenv:formatting]
-basepython = python3
+basepython = python3.8
 deps =
-    -r {toxinidir}/requirements_test_min.txt
     pre-commit~=2.16
 commands =
     pre-commit run --all-files
 
 [testenv]
 setenv =
     COVERAGE_FILE = {toxinidir}/.coverage.{envname}
-deps =
-    !pypy: -r {toxinidir}/requirements_test.txt
-    pypy: -r {toxinidir}/requirements_test_min.txt
+deps = poetry
 commands =
-    pytest {toxinidir}/tests/ {posargs:}
+    poetry install
+    poetry run pytest {toxinidir}/tests/ {posargs:}
 
 [testenv:coverage-html]
 setenv =
     COVERAGE_FILE = {toxinidir}/.coverage
-deps =
-    -r {toxinidir}/requirements_test.txt
+deps = poetry
 skip_install = true
 commands =
-    coverage combine
-    coverage html --ignore-errors --rcfile={toxinidir}/.coveragerc
+    poetry install
+    poetry run coverage combine
+    poetry run coverage html --ignore-errors --rcfile={toxinidir}/.coveragerc