Author rhettinger
Recipients mdk, ned.deily, pablogsal, rhettinger, vstinner
Date 2021-04-16.03:32:18
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1618543938.96.0.623423574105.issue42238@roundup.psfhosted.org>
In-reply-to
Content
If reactivated, the tool needs to be substantially improved.  It is NOT smart.  The false positives for slicing and logging examples are unnecessarily annoying.  It creates a barrier for people submitting documentation patches.  Each of the 367 entries in the susp-ignored file represents wasted time for contributors.

Also the CSV format is arcane, hard-to-read, and hard-to-edit.  It looks like it was quickly thrown together by someone who didn't care about usability.  Perhaps there should be a simpler tools that says, "take this current failure and mark it as a false positive" without trying to be over specific.

Mandatory checks with a high false positive rate are an anti-pattern for CI systems.  Already we've had one case of a contributor (me) who abandoned a doc patch rather than fight this tooling.
History
Date User Action Args
2021-04-16 03:32:18rhettingersetrecipients: + rhettinger, vstinner, ned.deily, mdk, pablogsal
2021-04-16 03:32:18rhettingersetmessageid: <1618543938.96.0.623423574105.issue42238@roundup.psfhosted.org>
2021-04-16 03:32:18rhettingerlinkissue42238 messages
2021-04-16 03:32:18rhettingercreate