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

Update the simple server example code for py3 modern syntax #21

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

adiroiban
Copy link
Member

This updates the code for py3 modern syntax

This is just a copy from #20 (comment)

@netlify
Copy link

netlify bot commented Aug 23, 2022

Deploy Preview for twisted-pr-preview ready!

Name Link
🔨 Latest commit 967d7dc
🔍 Latest deploy log https://app.netlify.com/sites/twisted-pr-preview/deploys/6320275107565f0008637452
😎 Deploy Preview https://deploy-preview-21--twisted-pr-preview.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

index.html Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
@adiroiban
Copy link
Member Author

Note that I have created this PR as an example of how you can contribute to the twisted web page.

I will let others suggest "the right(tm)" examples and do the review.

@graingert
Copy link
Member

graingert commented Sep 13, 2022

@adiroiban I actually copy pasted the snippet to make sure it worked, and found a few bugs, I also ran it through isort and black. We might want to add blacken-docs when this is all rst-ied

@adiroiban
Copy link
Member Author

We can use just jinja to include external python file ... or HTML <object> tag https://developer.mozilla.org/en-US/docs/Web/HTML/Element/object

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants