New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document zipfile CLI #72699
Comments
Command-line interface of the zipfile module is supported long time, but it is not documented. Proposed patch documents it. It is based on the documentation of tarfile CLI. |
Serhiy, your patch seems to be good, you can merge it, I have tested it with sphinx and the documentation is fine. Thanks |
LGTM. |
Serhiy, this needs .. program:: zipfile directive specified before cmdoption. Without it :option:`-c` links to python's -c, rather than zipfile's. The -l and -e are linked correctly, but the option links would have be be changed for all three, i.e. to :option:`-c <zipfile -c>` |
Thanks SilentGhost! |
Apparently (haven’t tried myself) if you put “.. program:: zipfile” before the :option: invocations, it changes the default context and you don’t need the <zipfile . . .> bit. The text in general looks fine. One very minor thing: I would use a hyphen when “command line” is a modifier on another noun, i.e. command-line interface, command-line options. |
Yes, moving program directive just after "Command Line Interface" heading works fine. |
New changeset feb6e8678512 by Serhiy Storchaka in branch '2.7': New changeset b51bf32defb1 by Serhiy Storchaka in branch '3.5': New changeset 843538a4094b by Serhiy Storchaka in branch '3.6': New changeset 0c8ffa562f3a by Serhiy Storchaka in branch 'default': |
Applied both Martin's suggestion. Thank you all! |
Misc/NEWS
so that it is managed by towncrier #552Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: