Update doc8 for new tool support and more reliable output parsing.
Review Request #11562 — Created March 30, 2021 and submitted
This updates
Doc8Tool
to subclass the newerBaseTool
, and to use the
newer features it offers (dependencies, improved configurable file
extension handling, centralized base command building, and standardized
error code output).The new version of the tool has safer report parsing, using a regex
rather than manual string splitting and parsing to match lines. This
simplifies things quite a bit and avoids possible errors, or crashes if
doc8 changes output someday.Unit tests were added to cover the new doc8 parsing, and to provide
integration tests.
Unit tests pass on Python 2.7 and 3.x.
Summary | ID |
---|---|
5f4ed6d9035ee30738328a2e5f5e7259affa6789 |
Description | From | Last Updated |
---|---|---|
E251 unexpected spaces around keyword / parameter equals |
![]() |
|
E251 unexpected spaces around keyword / parameter equals |
![]() |
|
E251 unexpected spaces around keyword / parameter equals |
![]() |
|
E251 unexpected spaces around keyword / parameter equals |
![]() |
|
E251 unexpected spaces around keyword / parameter equals |
![]() |
|
E251 unexpected spaces around keyword / parameter equals |
![]() |
|
E251 unexpected spaces around keyword / parameter equals |
![]() |
|
E251 unexpected spaces around keyword / parameter equals |
![]() |
- Change Summary:
-
Fixed a bad copy/paste leading to a keyword argument in
keyword = value
form. - Commits:
-
Summary ID d232bea815a687e540f2b9abc8dce1b54cf4e1ab 3c71932cb016c7fc41d26abf242dfd8b70dc31d0