Skip to content

bpo-36651: Fixed Asyncio Event Loop documentation inconsistency #12866

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

Merged
merged 5 commits into from
Apr 18, 2019

Conversation

naccht
Copy link
Contributor

@naccht naccht commented Apr 17, 2019

bpo-36651: Fixed Asyncio Event Loop documentation inconsistency

In the documentation for the call_later and the call_at methods there is a note which says that the delay cannot be longer than a day, but both methods have a note saying that this limitation was removed in Python 3.8
Here I fixed this issue by removing the pre-exising note and added a versionchanged.

To test my changes I have rebuilt the documentation with make html. I did not have any errors and the effected page displayed correctly on a browser.

https://bugs.python.org/issue36651

@naccht naccht requested review from 1st1 and asvetlov as code owners April 17, 2019 20:12
@the-knights-who-say-ni
Copy link

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed the PSF contributor agreement (CLA).

Our records indicate we have not received your CLA. For legal reasons we need you to sign this before we can look at your contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

If you have recently signed the CLA, please wait at least one business day
before our records are updated.

You can check yourself to see if the CLA has been received.

Thanks again for your contribution, we look forward to reviewing it!

@@ -0,0 +1 @@
bpo-36651: Fixed Asyncio Event Loop documentation inconsistency
Copy link
Member

Choose a reason for hiding this comment

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

There is an error in Travis due to this blurb entry at https://travis-ci.org/python/cpython/jobs/521431229#L309 . I am not sure this doc change needs a news entry.

blurb.BlurbError: Error in Misc/NEWS.d/next/Documentation/2019-04-17-20-21-32.bpo-36651.3iUjvY.rst:0:
Blurb 'body' can't start with 'bpo-'!

Removing file not passing bot checking
The change is small and self-descriptive. There is no need for changelog record in this case
@miss-islington
Copy link
Contributor

Sorry, I can't merge this PR. Reason: Base branch was modified. Review and try the merge again..

@miss-islington miss-islington merged commit 7e954e7 into python:master Apr 18, 2019
@miss-islington
Copy link
Contributor

Thanks @naccht for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Apr 18, 2019
…onGH-12866)

GH- [bpo-36651](https://bugs.python.org/issue36651): Fixed Asyncio Event Loop documentation inconsistency

In the documentation for the call_later and the call_at methods there is a note which says that the delay cannot be longer than a day, but both methods have a note saying that this limitation was removed in Python 3.8
Here I fixed this issue by removing the pre-exising note and added a versionchanged.

To test my changes I have rebuilt the documentation with ```make html```. I did not have any errors and the effected page displayed correctly on a browser.

https://bugs.python.org/issue36651
(cherry picked from commit 7e954e7)

Co-authored-by: Enrico Alarico Carbognani <[email protected]>
@bedevere-bot
Copy link

GH-12874 is a backport of this pull request to the 3.7 branch.

miss-islington added a commit that referenced this pull request Apr 18, 2019
…2866)

GH- [bpo-36651](https://bugs.python.org/issue36651): Fixed Asyncio Event Loop documentation inconsistency

In the documentation for the call_later and the call_at methods there is a note which says that the delay cannot be longer than a day, but both methods have a note saying that this limitation was removed in Python 3.8
Here I fixed this issue by removing the pre-exising note and added a versionchanged.

To test my changes I have rebuilt the documentation with ```make html```. I did not have any errors and the effected page displayed correctly on a browser.

https://bugs.python.org/issue36651
(cherry picked from commit 7e954e7)

Co-authored-by: Enrico Alarico Carbognani <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants