bpo-42299: Remove formatter module#23476
Conversation
vstinner
left a comment
There was a problem hiding this comment.
LGTM. I'm fine with removing the module. I just left minor remarks on the documentation, you're free to ignore them.
terryjreedy
left a comment
There was a problem hiding this comment.
LGTM. I simply don't know if anything else is needed.
terryjreedy
left a comment
There was a problem hiding this comment.
See suggested change, which applies to news also.
|
When you're done making the requested changes, leave the comment: |
Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
vstinner
left a comment
There was a problem hiding this comment.
LGTM. Thanks @terryjreedy for the way better documentation than what I wrote :-) It's good to write down the full rationale, otherwise people get angry when they see that we remove modules on purpose.
|
@vstinner @terryjreedy |
|
Victor, I was inspired to add a 'solution' by your sentence from issue 39352: "I'm no longer sure that it's a good idea to remove a module from the stdlib without providing a solution for the few people relying on it." In this case, I am convinced that copying is a better solution than putting the module on PyPI. |
https://bugs.python.org/issue42299