notifications.py missing several docstrings

Bug #1410922 reported by Tom Cameron
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Low
David Stanek

Bug Description

While reviewing notifications.py I noticed several docstrings were missing. An example can be found at the definition of register_event_callback.

Tom Cameron (drdabbles)
Changed in keystone:
assignee: nobody → Tom Cameron (tom-cameron)
Dolph Mathews (dolph)
Changed in keystone:
importance: Undecided → Low
Changed in keystone:
status: New → In Progress
Changed in keystone:
milestone: none → kilo-rc1
Revision history for this message
Lance Bragstad (lbragstad) wrote :

This is the review that should close out this bug:

https://review.openstack.org/#/c/147313/

Changed in keystone:
assignee: Tom Cameron (tom-cameron) → David Stanek (dstanek)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/147313
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=b5f326ce1cec71a906b56148d6106a407e96dd0a
Submitter: Jenkins
Branch: master

commit b5f326ce1cec71a906b56148d6106a407e96dd0a
Author: Tom Cameron <email address hidden>
Date: Wed Jan 14 15:48:46 2015 -0600

    Add docstrings to keystone.notifications functions

    Add docstrings to several functions that had none.
    Provide :param and :returns where appropriate
    Cleanup docstrings with :type

    Closes-Bug: 1410922
    Change-Id: Ib08f5486d2701d145d664badca4e9c86638e02d1

Changed in keystone:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in keystone:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: kilo-rc1 → 2015.1.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.