The latest flake8 version expects the keywords to have no parenthesis (e.g. assert foo vs assert(foo) or a space (e.g. assert (foo)) if needed.
Details
Details
- Reviewers
PiRK - Group Reviewers
Restricted Project - Commits
- rABCc1a376159d18: [linter] Make flake8 version >= 5 happy
With flake8 version 5.0.0:
arc lint --everything ninja all check-all
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
LGTM, with maybe one nit
| test/functional/p2p_getaddr_caching.py | ||
|---|---|---|
| 84 ↗ | (On Diff #34806) | This is the one case where assert (...) would be slightly better, as the parentheses would allow splitting the following statement over multiple lines and not cause autopep8 to introduce its awkward auto-formatting. |