Upgrade to Poetry v1.2.0 and other related cleanup

.github/workflows/tox.yml

@@ -1,3 +1,4 @@
+# vim: set ft=yaml ts=2 sw=2:
 name: Test Suite
 on:
   push:
@@ -10,46 +11,81 @@ on:
     - cron: '05 17 15 * *'  # 15th of the month at 5:05pm UTC
 jobs:
   build:
-    runs-on: ubuntu-latest
     strategy:
       matrix:
         os: [ubuntu-latest, macos-latest]
         python: ["3.7", "3.8", "3.9", "3.10" ]
+    runs-on: ${{ matrix.os }}
+    env:
+      # Run builds using a specific version of Poetry, to avoid surprises
+      POETRY_VERSION: "1.2.0"
+      # Prevent Poetry v1.2.0 from using the Python keyring, which sometimes fails or hangs on Linux
+      # See: https://github.com/python-poetry/poetry/issues/2692#issuecomment-1235683370
+      PYTHON_KEYRING_BACKEND: "keyring.backends.null.Keyring"
     steps: 
       - name: Check out code
         uses: actions/checkout@v2
       - name: Setup Python
         uses: actions/setup-python@v2
         with:
           python-version: ${{ matrix.python }}
-      - name: Install and configure Poetry
+      - name: Platform information
+        shell: bash
+        run: |
+          # Show platform information
+          python -c "import sys;print('Python %s' % sys.version)"
+      - name: Install Poetry
         uses: snok/install-poetry@v1.2.1 # see https://github.com/snok/install-poetry
         with:
+          version: ${{ env.POETRY_VERSION }}
           virtualenvs-create: true
           virtualenvs-in-project: true
+      - name: Install Poetry plugins
+        shell: bash
+        run: |
+          # Install Poetry plugins
+          poetry self add poetry-plugin-export
+      - name: Cache Poetry dependencies
+        uses: actions/cache@v2
+        with:
+          path: .venv
+          key: venv-${{ runner.os }}-${{ matrix.python }}-${{ hashFiles('**/poetry.lock') }}
       - name: Install dependencies
-        run: poetry install -v
+        shell: bash
+        run: |
+          # Install dependencies
+          poetry install --sync
       - name: Upgrade embedded tools within virtualenv
-        run: poetry run pip install --upgrade pip setuptools wheel
+        shell: bash
+        run: |
+          # Upgrade embedded tools
+          poetry run pip install --upgrade pip setuptools wheel
       - name: Run Tox test suite
+        shell: bash
         run: |
+          # Run Tox test suite
           poetry run which supybot-test  # this is the only way to run the true Limnoria tests
           SUPYBOT_TEST=$(poetry run which supybot-test) poetry run tox -c .toxrc -e "checks,coverage"
       - name: Upload coverage data to coveralls.io
-        run: poetry run coveralls --service=github
+        shell: bash
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           COVERALLS_FLAG_NAME: ${{ runner.os }}-python${{ matrix.python }}
           COVERALLS_PARALLEL: true
+        run: | 
+          # Upload coverage data to coveralls.io
+          poetry run coveralls --service=github
   coveralls:
     name: Indicate completion to coveralls.io
     needs: build
     runs-on: ubuntu-latest
     container: python:3-slim
     steps:
     - name: Finished
-      run: |
-         pip3 install --upgrade coveralls
-         coveralls --service=github --finish
+      shell: bash
       env:
         GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+      run: |
+        # Indicate completion to coveralls.io
+        pip3 install --upgrade coveralls
+        coveralls --service=github --finish

.pre-commit-config.yaml

@@ -4,7 +4,7 @@ repos:
     hooks:
       - id: system
         name: Requirements
-        entry: poetry export --format=requirements.txt --without-hashes --dev --output=docs/requirements.txt
+        entry: poetry export --format=requirements.txt --without-hashes --with dev --output=docs/requirements.txt
         pass_filenames: false
         language: system
   - repo: local

.pylintrc

@@ -70,11 +70,9 @@ disable=fixme,
         missing-function-docstring,
         unused-wildcard-import,
         trailing-whitespace,
-        bad-continuation,
         too-few-public-methods,
         no-else-return,
         no-else-raise,
-        no-self-use,
         duplicate-code,
         consider-using-f-string,
         not-an-iterable, # gets confused with lists on attrs classes; MyPy covers this
@@ -273,13 +271,6 @@ max-line-length=132
 # Maximum number of lines in a module.
 max-module-lines=1000
 
-# List of optional constructs for which whitespace checking is disabled. `dict-
-# separator` is used to allow tabulation in dicts, etc.: {1  : 1,\n222: 2}.
-# `trailing-comma` allows a space between comma and closing bracket: (a, ).
-# `empty-line` allows space-only lines.
-no-space-check=trailing-comma,
-               dict-separator
-
 # Allow the body of a class to be on the same line as the declaration if body
 # contains single statement.
 single-line-class-stmt=no

.toxrc

@@ -7,14 +7,14 @@ envlist =
 ignore_basepython_conflict = true
 
 [testenv:checks]
-whitelist_externals = poetry
+allowlist_externals = poetry
 commands =
    poetry --version
    poetry version
    poetry run pre-commit run --all-files --show-diff-on-failure
 
 [testenv:coverage]
-whitelist_externals = poetry
+allowlist_externals = poetry
 passenv = SUPYBOT_TEST
 commands =
    poetry --version
@@ -25,7 +25,7 @@ commands =
    poetry run coverage report
 
 [testenv:docs]
-whitelist_externals = poetry
+allowlist_externals = poetry
 changedir = docs
 commands =
    poetry --version

Changelog

@@ -1,3 +1,12 @@
+Version 0.6.2     unreleased
+
+	* Upgrade to Poetry v1.2.0 and make related build process changes.
+	* Upgrade to Pylint v2.15.0 and address warnings.
+	* Fix Pylint's configured class-attribute-naming-style to be snake_case.
+	* Adjust Black configuration to target Python 3.7, 3.8, 3.9, and 3.10.
+	* Add dependency caching to GitHub Actions workflow.
+	* Fix the GitHub Actions matrix build to properly target MacOS.
+
 Version 0.6.1     09 Jun 2022
 
 	* Modernize usage of attrs and cattrs.

DEVELOPER.md

@@ -25,7 +25,7 @@ suites and combines the coverage results together into a single report.
 
 ## Packaging and Dependencies
 
-This project uses [Poetry](https://python-poetry.org/) to manage Python packaging and dependencies.  Most day-to-day tasks (such as running unit tests from the command line) are orchestrated through Poetry.  
+This project uses [Poetry](https://python-poetry.org/) to manage Python packaging and dependencies.  Most day-to-day tasks (such as running unit tests from the command line) are orchestrated through Poetry.
 
 A coding standard is enforced using [Black](https://github.com/psf/black), [isort](https://pypi.org/project/isort/) and [Pylint](https://www.pylint.org/).  Python 3 type hinting is validated using [MyPy](https://pypi.org/project/mypy/).  To reduce boilerplate, classes are defined using [Attrs](https://www.attrs.org/) (see this [rationale](https://glyph.twistedmatrix.com/2016/08/attrs.html)).
 
@@ -67,25 +67,47 @@ of this writing.
 Nearly all prerequisites are managed by Poetry.  All you need to do is make
 sure that you have a working Python 3 enviroment and install Poetry itself.
 
+### Poetry Version
+
+The project is designed to work with Poetry >= 1.2.0.  If you already have an older
+version of Poetry installed on your system, uninstall it before following the setup
+process below:
+
+```
+curl -sSL https://raw.githubusercontent.com/python-poetry/poetry/master/get-poetry.py | python3 - --uninstall
+```
+
+See the [Announcing Poetry 1.2.0](https://python-poetry.org/blog/announcing-poetry-1.2.0/)
+blog post for more information.
+
 ### MacOS
 
-On MacOS, it's easiest to use [Homebrew](https://brew.sh/):
+On MacOS, it's easiest to use [Homebrew](https://brew.sh/) to install Python:
 
 ```
-$ brew install python3
-$ brew install poetry
+brew install python3
 ```
 
 Once that's done, make sure the `python` on your `$PATH` is Python 3 from
 Homebrew (in `/usr/local`), rather than the standard Python 2 that comes with
 MacOS.
 
+Although Poetry can also be installed from Homebrew, it works better to use
+to [official installer](https://python-poetry.org/docs/#installing-with-the-official-installer):
+
+```
+curl -sSL https://install.python-poetry.org | python3 -
+```
+
+> _Note:_ The installer prints the location of the installed `poetry` script.
+> Make sure to add this to your `$PATH`, otherwise you won't be able to run it.
+
 ### Debian
 
 First, install Python 3 and related tools:
 
 ```
-$ sudo apt-get install python3 python3-venv python3-pip
+sudo apt-get install python3 python3-venv python3-pip
 ```
 
 Next, make sure that the `python` interpreter on your `$PATH` is Python 3.
@@ -96,21 +118,23 @@ However, by default there is only a `python3` interpreter on your `$PATH`, not
 a `python` interpreter.  To add the `python` interpreter, use:
 
 ```
-$ sudo apt-get install python-is-python3
+sudo apt-get install python-is-python3
 ```
 
 For earlier releases of Debian where both Python 2 and Python 3 are available,
 the process is a little more complicated.  The approach I used before upgrading
 to _bullseye_ was based on `update-alternatives`, as discussed on
 [StackExchange](https://unix.stackexchange.com/a/410851).
 
-Once Python 3 is on your `$PATH` as `python`, install Poetry in your home
-directory:
+Next, install Poetry using the [official installer](https://python-poetry.org/docs/#installing-with-the-official-installer):
 
 ```
-$ curl -sSL https://install.python-poetry.org | python3 -
+curl -sSL https://install.python-poetry.org | python3 -
 ```
 
+> _Note:_ The installer prints the location of the installed `poetry` script.
+> Make sure to add this to your `$PATH`, otherwise you won't be able to run it.
+
 ## Developer Tasks
 
 The [`run`](run) script provides shortcuts for common developer tasks:
@@ -125,7 +149,6 @@ Shortcuts for common developer tasks
 Usage: run <command>
 
 - run install: Setup the virtualenv via Poetry and install pre-commit hooks
-- run activate: Print command needed to activate the Poetry virtualenv
 - run requirements: Regenerate the docs/requirements.txt file
 - run format: Run the code formatters
 - run checks: Run the code checkers
@@ -150,27 +173,27 @@ works well and there are are Debian-specific install [instructions](https://wiki
 you need more help:
 
 ```
-$ sudo apt-get install inspircd
+sudo apt-get install inspircd
 ```
 
 Next, install an IRC client.  Any client is ok, but [Irssi](https://irssi.org/) works well:
 
 ```
-$ sudo apt-get install irssi
+sudo apt-get install irssi
 ```
 
 Once the IRC server is up, make sure you can connect with the client:
 
 ```
-$ irssi --nick=ken --connect=localhost
+irssi --nick=ken --connect=localhost
 ```
 
 Once you are connected, join the testing channel with `/join #localtest`.
 
 Finally, open another terminal window and run the bot:
 
 ```
-$ run bot
+run bot
 Running the local bot...
 INFO 2021-02-14T17:06:34 Connecting to localhost:6667.
 WARNING 2021-02-14T17:06:34 Error connecting to localhost:6667: ConnectionRefusedError: [Errno 111] Connection refused
@@ -226,7 +249,7 @@ order.  In particular, if you do not run the install step, there will be no
 virtualenv for PyCharm to use:
 
 ```
-$ run install && run checks && run test
+run install && run checks && run test
 ```
 
 ### Open the Project
@@ -384,7 +407,7 @@ Version 0.1.29     unreleased
 Run the release step:
 
 ```
-$ run release 0.1.29
+run release 0.1.29
 ```
 
 This updates `pyproject.toml` and the `Changelog` to reflect the released
@@ -395,7 +418,7 @@ and revert your commit (`git reset HEAD~1`) if you made a mistake.
 Finally, publish the release:
 
 ```
-$ run publish
+run publish
 ```
 
 This builds the deployment artifacts, publishes the artifacts to PyPI, and