Page MenuHomePhabricator

Check for unused imports in *.py files
ClosedPublic

Authored by Fabien on Jan 30 2019, 14:58.

Details

Summary

Using flake8 code F401, unused imports are output as lint errors from
acanist.
flake8 is preferred here over pylint because it offers more feature for
the long-term improvement. It could also throw pep8 errors but will not
fix them like autopep8 does, so better use both.

Based on core PR11835

Depends on D2445, D2446, D2447, D2448, D2449, D2450

Test Plan

Add an unused import in a py file and run arc lint. An error should be
reported
Read the CONTRIBUTING.md doc

Diff Detail

Repository
rABC Bitcoin ABC
Branch
flake8
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4772
Build 7607: Bitcoin ABC Buildbot (legacy)
Build 7606: arc lint + arc unit