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 rhettinger
Recipients Greg Price, benjamin.peterson, ezio.melotti, rhettinger
Date 2019-09-10.19:31:45
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1568143905.98.0.71913661352.issue38043@roundup.psfhosted.org>
In-reply-to
Content
This is mostly harmless but I'm concerned that we're encouraging a new Python developer to:

* churn code in mostly minor ways, irrelevant to users

* altering code long known to be stable, increasing
  the risk of introducing new bugs or performance changes

* altering code in ways that are atypical for our 
  code base (i.e. the bool type isn't a norm in our
  code, we mostly use int for that)

* altering code without communicating with the developer
  who originally wrote that code (if they are still active)

* consuming the time of reviewers when they could be working
  on known bugs, legitimate feature requests, or documentation

* one-off or drive-by code alterations rather that what
  Guido calls "holistic refactoring" where we do clean-ups
  while understanding and thinking about the module as a
  whole and focusing on the user experience.

* unfortunately, making lots of random, minor changes to
  a code base in a major project is an addictive experience
  and IMO it would be best to re-channel it early, particularly
  if the changes are motivated by "I like my style of coding
  more than that of the original contributor".  Style changes
  are highly subjective and usually we defer to the original
  contributor who was closest to the problem being solved.
History
Date User Action Args
2019-09-10 19:31:46rhettingersetrecipients: + rhettinger, benjamin.peterson, ezio.melotti, Greg Price
2019-09-10 19:31:45rhettingersetmessageid: <1568143905.98.0.71913661352.issue38043@roundup.psfhosted.org>
2019-09-10 19:31:45rhettingerlinkissue38043 messages
2019-09-10 19:31:45rhettingercreate