Opened 5 years ago

Closed 5 years ago

#4439 enhancement closed fixed (fixed)

Documentation: clarify that callback() may return before callback chain is complete

Reported by: jrennie Owned by:
Priority: normal Milestone:
Component: core Keywords: documentation
Cc: Branch: branches/callback-chaining-doc-4439
(diff, github, buildbot, log)
Author: exarkun Launchpad Bug:

Description

This ticket simply requests that documentation for the callback and errback methods of twisted.internet.defer.Deferred clarify when they will return. As currently written, a programmer might assume that these methods will not return until the chain has completed. In fact, the methods will return as soon as they run into a Deferred-returning function in the chain. 'course, it's probably best to provide no guarantees as to when these functions will return in case the Twisted implementation changes.

Change History (7)

comment:1 Changed 5 years ago by exarkun

  • Author set to exarkun
  • Branch set to branches/callback-chaining-doc-4439

(In [29049]) Branching to 'callback-chaining-doc-4439'

comment:2 Changed 5 years ago by exarkun

(In [29050]) Expand Deferred.callback docstring

refs #4439

comment:3 Changed 5 years ago by exarkun

  • Keywords documentation review added
  • Owner glyph deleted

comment:4 Changed 5 years ago by ivank

  • Keywords review removed

Very minor nit: *error*- callback should be *error*-callback. Everything else looks good, please merge.

comment:5 Changed 5 years ago by ivank

  • Owner set to exarkun

comment:6 Changed 5 years ago by exarkun

  • Resolution set to fixed
  • Status changed from new to closed

(In [29113]) Merge callback-chaining-doc-4439

Author: exarkun
Reviewer: ivank
Fixes: #4439

Augment the Deferred.callback docstring with text about chaining.

comment:7 Changed 4 years ago by <automation>

  • Owner exarkun deleted
Note: See TracTickets for help on using tickets.