This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author taleinat
Recipients Mariatta, dfrojas, eli.bendersky, nedbat, rhettinger, scoder, serhiy.storchaka, taleinat, vstinner
Date 2018-11-10.18:37:10
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1541875030.66.0.788709270274.issue34160@psf.upfronthosting.co.za>
In-reply-to
Content
>> ISTM that the coverage tests as currently written aren't good tests.

> Hi, I'd like to remind everyone to be open, respectful, and considerate. There are ways to describe hos things that can be improved. There is no need to denigrate other people's work.

I find this to be an overreaction in this case.  Sure, it could have been worded more positively, but the negativity was very mild; the tests weren't even being called "bad", not to mention overly negative wording e.g. "horrible".

Further, you omitted the followup explanation of *what about the tests isn't good*:

> Otherwise, the tests are relying on a non-guaranteed implementation detail.

IMO we shouldn't require ourselves to be overly careful in our wording, such as avoiding any negative wording entirely.
History
Date User Action Args
2018-11-10 18:37:10taleinatsetrecipients: + taleinat, rhettinger, scoder, vstinner, nedbat, eli.bendersky, serhiy.storchaka, Mariatta, dfrojas
2018-11-10 18:37:10taleinatsetmessageid: <1541875030.66.0.788709270274.issue34160@psf.upfronthosting.co.za>
2018-11-10 18:37:10taleinatlinkissue34160 messages
2018-11-10 18:37:10taleinatcreate