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.

Sign up for GitHub

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

Jump to bottom

Old style coroutines used the readthedocs #436

Closed
Pradhvan opened this issue Sep 11, 2019 · 1 comment · Fixed by #437
Closed

Old style coroutines used the readthedocs #436

Pradhvan opened this issue Sep 11, 2019 · 1 comment · Fixed by #437

Comments

@Pradhvan
Copy link

Hi! I was going through the documentation and found out that the restructuredtext file still has the old-style coroutines instead of the async/await keyword, is there a particular reason for that?

If not I can submit a PR to change the code in the readthedocs to the new async/await feature.
@asvetlov
Copy link
Member

Please do, excellent idea!

Pradhvan pushed a commit to Pradhvan/aiomysql that referenced this issue Sep 11, 2019
@Glados567
docs/index.rst: Updated tutorial to native coroutines
7d3019b 
This updates the old styled @asyncio.coroutine -> async def coroutine and updates yield from -> await keyword.

Closes aio-libs#436
Pradhvan pushed a commit to Pradhvan/aiomysql that referenced this issue Sep 11, 2019
@Glados567
docs/index.rst: Updated tutorial to native coroutines
bec31e3 
This updates the old styled @asyncio.coroutine -> async def coroutine and updates yield from -> await.

Closes aio-libs#436
@Pradhvan Pradhvan mentioned this issue Sep 11, 2019
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs/index.rst: Updated tutorial to native coroutines Pradhvan/aiomysql
2 participants
@asvetlov @Pradhvan