Opened 9 years ago

Closed 9 years ago

#6272 enhancement closed fixed (fixed)

Minor grammar blooper in "Deferred Reference"

Reported by: Hynek Schlawack Owned by: Tom Prince
Priority: normal Milestone:
Component: core Keywords: documentation
Cc: Branch: branches/defer-typo-6272
branch-diff, diff-cov, branch-cov, buildbot
Author: hynek, tom.prince

Description

"If the errback does returns a Failure or raise an exception,"

There’s a "does" too much and an "s" missing.

Attachments (1)

typo-in-defer.patch (824 bytes) - added by Hynek Schlawack 9 years ago.

Download all attachments as: .zip

Change History (7)

Changed 9 years ago by Hynek Schlawack

Attachment: typo-in-defer.patch added

comment:1 Changed 9 years ago by Tom Prince

Author: tomprince
Branch: branches/defer-typo-6272

(In [36873]) Branching to defer-type-6272.

comment:2 Changed 9 years ago by Tom Prince

Author: tomprincehynek
Keywords: review removed
Owner: set to Tom Prince
Status: newassigned

I think the fix in the patch is backwards. The does is intentional.

comment:3 Changed 9 years ago by Tom Prince

Author: hynekhynek, tom.prince
Keywords: review added
Owner: Tom Prince deleted
Status: assignednew

comment:4 Changed 9 years ago by Jean-Paul Calderone

Owner: set to Jean-Paul Calderone
Status: newassigned

(In [36873]) Branching to defer-type-6272.

Note this is not actually the branch name :/

comment:5 Changed 9 years ago by Jean-Paul Calderone

Keywords: review removed
Owner: changed from Jean-Paul Calderone to Tom Prince
Status: assignednew

I think the fix in the patch is backwards. The does is intentional.

I agree. I think the fix in the branch to make the "does return" agreement is correct. Please merge (with a misc fragment).

comment:6 Changed 9 years ago by Tom Prince

Resolution: fixed
Status: newclosed

(In [36877]) Merge branches/defer-typo-6272: Fix minor typo in "Deferred Reference"

Author: hynek, tom.prince Reviewers: tom.prince, exarkun Fixes: #6272

Note: See TracTickets for help on using tickets.