Fixes the warnings listed here: https://github.com/adamchainz/flake8-comprehensions#rules
Also remove two unused imports in dnssec.py to fix flake8 error introduced in D14094 when I deleted the code using these imports.
Depends on D14106
Details
Details
- Reviewers
Fabien - Group Reviewers
Restricted Project - Commits
- rABC0906a07d43a9: [electrum] fix flake8-comprehensions warnings and enable flake8 for electrum
arc lint --everything cd electrum python -m electrumabc.tests python -m electrumabc_plugins.fusion.tests
Run electrum-abc, enable cash fusion.
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| electrum/electrumabc/paymentrequest.py | ||
|---|---|---|
| 273 ↗ | (On Diff #40939) | here x is a namedtuple TxOutput, defined in transaction.py as: class TxOutput(NamedTuple):
type: int
destination: DestinationType
# str when the output is set to max: '!'
value: Union[int, str]So we can call the value by attribute name rather than tuple index. |
| electrum/electrumabc_gui/stdio.py | ||
| 188 ↗ | (On Diff #40939) | There seems to be an inconsistency in the formatting of the header line, here, but this is out of the scope of this diff. I'm not quite sure how to test this code, it may even be unused. |