]> git.ipfire.org Git - thirdparty/sqlalchemy/alembic.git/commit
ensure single import per line
authorMike Bayer <mike_mp@zzzcomputing.com>
Tue, 28 Feb 2023 15:58:22 +0000 (10:58 -0500)
committerMike Bayer <mike_mp@zzzcomputing.com>
Tue, 28 Feb 2023 16:37:02 +0000 (11:37 -0500)
commit919297555f17efedcdcc7cc936204a1b86b804d3
tree45f92ba2c4428a0f3ab99bda5cf9a6a8e662f8e8
parentb53ec0004a08c40a25a4dbf047c51cd140971a9c
ensure single import per line

This adds the very small plugin flake8-import-single which
will prevent us from having an import with more than one symbol
on a line.

Flake8 by itself prevents this pattern with E401:

   import collections, os, sys

However does not do anything with this:

   from sqlalchemy import Column, text

Both statements have the same issues generating merge artifacts
as well as presenting a manual decision to be made.   While
zimports generally cleans up such imports at the top level, we
don't enforce zimports / pre-commit use.

the plugin finds the same issue for imports that are inside of
test methods.   We shouldn't usually have imports in test methods
so most of them here are moved to be top level.

The version is pinned at 0.1.5; the project seems to have no
activity since 2019, however there are three 0.1.6dev releases
on pypi which stopped in September 2019, they seem to be
experiments with packaging.  The source for 0.1.5
is extremely simple and only reveals one method to flake8
(the run() method).

Change-Id: Icea894e43bad9c0b5d4feb5f49c6c666d6ea6aa1
.pre-commit-config.yaml
alembic/testing/env.py
tests/test_autogen_render.py
tests/test_op.py
tests/test_postgresql.py
tests/test_script_production.py
tox.ini