[issue42238] Deprecate suspicious.py?
Julien Palard
report at bugs.python.org
Sun Nov 1 18:49:23 EST 2020
New submission from Julien Palard <julien+python at palard.fr>:
I was not here 21 years ago when it was introduced [1], but according to the commit message it was introduced to find leftover Latex mardown.
It tries to find 4 patterns in Sphinx node text (not in raw rst files):
::(?=[^=])| # two :: (but NOT ::=)
This one has ~100 false positive in susp-ignored.csv (pypi classifiers, slices, ipv6, ...)
:[a-zA-Z][a-zA-Z0-9]+| # :foo
This one has ~300 false positive in susp-ignored.csv (slices, C:\, ipv6, ...)
`| # ` (seldom used by itself)
This one has ~20 false positive in susp-ignored.csv (mostly reStructuredText in code-blocks)
(?<!\.)\.\.[ \t]*\w+: # .. foo: (but NOT ... else:)
This one does not have false positives.
The script, on my laptop (with a core i9), is slow (4mn20s), and it's probably way slower on the CI.
I tried to search for `suspicious is:pr in:comments` on github to see if it's usefull:
- 2 contributor had an issue with the script (gh-9748, gh-21940)
- 5 had to add false positive to susp-ignored.csv (gh-20556, gh-13772, gh-11481, gh-9317, gh-6915)
- 4 had to update susp-ignored.csv (gh-11769, gh-5552, gh-3694, gh-2719)
- 1 did not addedd to susp-ignored but changed to avoid a false positive (gh-18939)
Case where it actually helped:
- Finding an error: (gh-12562 .. literalinclude: instead of .. literalinclude::)
- Finding refs in code block (gh-7413)
- Writing plaintext in Misc/NEWS (gh-1339)
I'd go for enhancing rstlint.py (which is fast, ~1s on my laptop) a bit to try to handle the `.. literalinclude:` missing a `:` errors, and dropping suspicious.
So I'd appreciate feedback on this script, did it helped you recently?
1: https://github.com/python/cpython/commit/700cf28f410521066f40671f1da7db0302d753fd
----------
assignee: mdk
components: Documentation
messages: 380167
nosy: mdk
priority: normal
severity: normal
status: open
title: Deprecate suspicious.py?
_______________________________________
Python tracker <report at bugs.python.org>
<https://bugs.python.org/issue42238>
_______________________________________
More information about the Python-bugs-list
mailing list