Skip to content

bpo-36345: Add a new example in the documentation of wsgiref - #12511

Merged
brettcannon merged 2 commits into
python:masterfrom
matrixise:bpo-36345-part-2
Mar 25, 2019
Merged

bpo-36345: Add a new example in the documentation of wsgiref#12511
brettcannon merged 2 commits into
python:masterfrom
matrixise:bpo-36345-part-2

Conversation

@matrixise

@matrixisematrixise commented Mar 23, 2019

Copy link
Copy Markdown
Member

It's the second PR where I convert the code of Tools/scripts/serve.py as an example in the documentation of wsgiref.

https://bugs.python.org/issue36345

@eamanueamanu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm. IMO should be good tell how the example is executed.

Comment threadDoc/library/wsgiref.rst Outdated
@brettcannon

Copy link
Copy Markdown
Member

One minor change and then we can merge this! (I tried to merge it myself but you didn't give repo owners edit rights on your PRs, @matrixise ).

Co-Authored-By: matrixise <stephane@wirtel.be>
@matrixise

Copy link
Copy Markdown
MemberAuthor

Thank you so much @brettcannon , About the rights, this is strange because I just create this PR and I think the flag was checked. really sorry.

@brettcannon
brettcannon merged commit 360e1e4 into python:masterMar 25, 2019
@bedevere-bot

Copy link
Copy Markdown

@brettcannon: Please replace # with GH- in the commit message next time. Thanks!

@brettcannon

Copy link
Copy Markdown
Member

Thanks!

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docsDocumentation in the Doc dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@matrixise@brettcannon@bedevere-bot@eamanu@the-knights-who-say-ni