diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2019-05-30 11:31:03 -0400 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2019-06-20 13:50:41 -0400 |
| commit | 190e0139e834e4271268652e058c280787ae69eb (patch) | |
| tree | 21e93907a58cd2f390f687ddc5e0c1da1eb25454 /test/ext/declarative | |
| parent | ff8e7732b9f656f8cea05544660c18d57dd37864 (diff) | |
| download | sqlalchemy-190e0139e834e4271268652e058c280787ae69eb.tar.gz | |
Enable F841
This is a very useful assertion which prevents unused variables
from being set up allows code to be more readable and sometimes
even more efficient. test suites seem to be where the most
problems are and there do not seem to be documentation examples
that are using this, or at least the linter is not taking effect
within rst blocks.
Change-Id: I2b3341d8dd14da34879d8425838e66a4b9f8e27d
Diffstat (limited to 'test/ext/declarative')
| -rw-r--r-- | test/ext/declarative/test_concurrency.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/ext/declarative/test_concurrency.py b/test/ext/declarative/test_concurrency.py index 42d0b13a4..ac1ae0aaf 100644 --- a/test/ext/declarative/test_concurrency.py +++ b/test/ext/declarative/test_concurrency.py @@ -43,7 +43,7 @@ class ConcurrentUseDeclMappingTest(fixtures.TestBase): # this is the failure mode, where B is being handled by # declarative and is in the registry but not mapped yet. result[0] = oe - except exc.InvalidRequestError as err: + except exc.InvalidRequestError: # if make_b() starts too slowly, we can reach here, because # B isn't in the registry yet. We can't guard against this # case in the library because a class can refer to a name that |