gh-93096: Remove -t and -v flags from pickle cli#131068
Conversation
Lib/test/test_pickle.py Outdated
| def load_tests(loader, tests, pattern): | ||
| tests.addTest(doctest.DocTestSuite()) | ||
| tests.addTest(doctest.DocTestSuite(pickle)) |
There was a problem hiding this comment.
Oh, this looks like a bugfix. It should be backported. Can you extract this change into a new PR? It's not directly related to CLI changes.
There was a problem hiding this comment.
Yes, sure, should i open new issue or can i send PR for the current issue?
vstinner left a comment
vstinner left a comment There was a problem hiding this comment.
Would it be possible to document the current command line interface (CLI) in Doc/library/pickle.rst? https://docs.python.org/dev/library/pickle.html
donbarbos commented Mar 11, 2025
I think it can be suggested in issue #130160 |
vstinner left a comment
vstinner left a comment There was a problem hiding this comment.
LGTM.
@serhiy-storchaka: Are you ok with these changes?
vstinner commented Mar 11, 2025
vstinner merged commit 425e0af into python:mainUh oh!
There was an error while loading. Please reload this page.
3 participants
./python -m doctest Lib/pickle.py -v