Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2256)

Issue 66430046: docs refactoring

Can't Edit
Can't Publish+Mail
Start Review
Created:
7 years, 4 months ago by yselivanov
Modified:
7 years, 4 months ago
Reviewers:
GvR
Visibility:
Public.

Description

docs refactoring

Patch Set 1 #

Total comments: 8

Patch Set 2 : v2 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+49 lines, -38 lines) Patch
M Doc/library/asyncio-eventloop.rst View 1 17 chunks +23 lines, -22 lines 0 comments Download
M Doc/library/asyncio-stream.rst View 7 chunks +7 lines, -7 lines 0 comments Download
M Doc/library/asyncio-sync.rst View 7 chunks +7 lines, -7 lines 0 comments Download
M Doc/library/asyncio-task.rst View 1 4 chunks +12 lines, -2 lines 0 comments Download

Messages

Total messages: 7
GvR
https://codereview.appspot.com/66430046/diff/1/Doc/library/asyncio-eventloop.rst File Doc/library/asyncio-eventloop.rst (right): https://codereview.appspot.com/66430046/diff/1/Doc/library/asyncio-eventloop.rst#newcode391 Doc/library/asyncio-eventloop.rst:391: which result is set to ``None`` on success. which ...
7 years, 4 months ago (2014-02-20 19:53:31 UTC) #1
yselivanov
https://codereview.appspot.com/66430046/diff/1/Doc/library/asyncio-eventloop.rst File Doc/library/asyncio-eventloop.rst (right): https://codereview.appspot.com/66430046/diff/1/Doc/library/asyncio-eventloop.rst#newcode391 Doc/library/asyncio-eventloop.rst:391: which result is set to ``None`` on success. On ...
7 years, 4 months ago (2014-02-20 20:32:14 UTC) #2
GvR
https://codereview.appspot.com/66430046/diff/1/Doc/library/asyncio-eventloop.rst File Doc/library/asyncio-eventloop.rst (right): https://codereview.appspot.com/66430046/diff/1/Doc/library/asyncio-eventloop.rst#newcode441 Doc/library/asyncio-eventloop.rst:441: returns a :class:`Future`. On 2014/02/20 20:32:14, yselivanov wrote: > ...
7 years, 4 months ago (2014-02-20 20:37:54 UTC) #3
yselivanov
https://codereview.appspot.com/66430046/diff/1/Doc/library/asyncio-eventloop.rst File Doc/library/asyncio-eventloop.rst (right): https://codereview.appspot.com/66430046/diff/1/Doc/library/asyncio-eventloop.rst#newcode441 Doc/library/asyncio-eventloop.rst:441: returns a :class:`Future`. > I went back and forth ...
7 years, 4 months ago (2014-02-20 20:46:45 UTC) #4
GvR
On 2014/02/20 20:46:45, yselivanov wrote: > https://codereview.appspot.com/66430046/diff/1/Doc/library/asyncio-eventloop.rst > File Doc/library/asyncio-eventloop.rst (right): > > https://codereview.appspot.com/66430046/diff/1/Doc/library/asyncio-eventloop.rst#newcode441 > ...
7 years, 4 months ago (2014-02-20 20:50:38 UTC) #5
yselivanov
> I suggest always writing "it is a coroutine" and stretching the meaning of > ...
7 years, 4 months ago (2014-02-20 21:08:48 UTC) #6
GvR
7 years, 4 months ago (2014-02-20 21:16:42 UTC) #7
LGTM. Poor Larry!
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b