Skip to content

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.

@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.

3 participants