Skip to content

gh-93096: Update and document pickletools CLI - #131273

Merged
hugovk merged 2 commits into
python:mainfrom
donbarbos:issue-93096-pickletools-update
Mar 17, 2025
Merged

gh-93096: Update and document pickletools CLI#131273
hugovk merged 2 commits into
python:mainfrom
donbarbos:issue-93096-pickletools-update

Conversation

@donbarbos

@donbarbos donbarbos commented Mar 15, 2025 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Contributor

@hugovk what do you think about setting the same behavior as we did in pickle?

I changed nargs from * to + and I added pickle_file as option to docs

📚 Documentation preview 📚: https://cpython-previews--131273.org.readthedocs.build/
@bedevere-app bedevere-app Bot mentioned this pull request Mar 15, 2025
Open
27 tasks
hugovk
hugovk reviewed Mar 17, 2025
View reviewed changes

@hugovk hugovk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, let's also update the headers, from:

  • Command line usage
    • Command line options
  • Programmatic Interface

To:

  • Command-line usage
    • Command-line options
  • Programmatic interface
Comment thread Doc/library/pickletools.rst Outdated
@donbarbos

donbarbos commented Mar 17, 2025

Copy link
Copy Markdown
Contributor Author

done 👍
@hugovk
hugovk enabled auto-merge (squash) March 17, 2025 09:12
@hugovk
hugovk merged commit 85c04f8 into python:main Mar 17, 2025
plashchynski pushed a commit to plashchynski/cpython that referenced this pull request Mar 17, 2025
@donbarbos @plashchynski
pythongh-93096: Update and document pickletools CLI (python#131273)
16ec128
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news

2 participants

@donbarbos @hugovk