[Numpy-discussion] what to clean up, what to leave as is

[Ralf Gommers]

Hi all,

It's great to see that people are jumping at the chance to clean up Python
2 support. I would however caution about overdoing it on other cleanups. As
a reminder, we normally do not want pure style PRs (e.g. PEP8 cleanups),
because they make the code history (git blame, commits on particular files,
etc.) harder to work with, have review overhead, and may introduce new bugs
for little gain.

Imho that same rationale applies to things like converting strings to
f-strings. There's of course some gray area, for example removing "from ...
import *" can guard against accidentally exposing new API, so can be
considered a valuable cleanup.

As a separate/additional point: numpy.distutils and numpy.f2py are largely
untested, PRs are hard to test locally because of platform-specific code,
and changes often introduce regressions. So even for some cleanups that are
okay for other files, please do not do them on those modules.

Cheers,
Ralf
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.python.org/pipermail/numpy-discussion/attachments/20200124/d944bcfc/attachment.html>