Skip to content
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

Add documentation for the zipapp distribution #11459

Merged
merged 4 commits into from
Sep 18, 2022
Merged

Conversation

pfmoore
Copy link
Member

@pfmoore pfmoore commented Sep 18, 2022

Documents the new zipapp distribution, noting that it is still experimental.

@pfmoore
Copy link
Member Author

pfmoore commented Sep 18, 2022

At the moment, this has the zipapp as one subheading under "supported methods". As the zipapp is currently experimental, is this OK? The alternative would be to put it under "Alternative methods", but then that looks a bit odd because the existing text under than heading has no subheading. I tried having it as a heading on the same level as "Supported Methods", but that looked wrong (too prominent).

Does anyone have a preference? Words is hard 🙁


then the currently active Python interpreter will be used.

```{note}
Copy link
Member

Choose a reason for hiding this comment

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

Suggestion: put the admonition at the top of the section or after the first paragraph, coz people skim 😅

Copy link
Member Author

Choose a reason for hiding this comment

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

lol, good point. Done.

@pfmoore
Copy link
Member Author

pfmoore commented Sep 18, 2022

Merging this, we can always revise it later 🙂

@pfmoore pfmoore merged commit fbb7f0b into pypa:main Sep 18, 2022
@pfmoore pfmoore deleted the zipapp_docs branch September 18, 2022 12:16
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants