twistedchecker reports missing documentation when the documentation comes from an interface

Bug #1132540 reported by Richard Wall
This bug affects 2 people
Affects Status Importance Assigned to Milestone

Bug Description

Often the api documentation for a class, method or function comes from a zope.interface in which case the docstring can be omitted.

twistedchecker should handle this situation.


************* Module twisted.names.common
W9208: 66,4:ResolverBase.query: Missing docstring
W9208: 80,4:ResolverBase.lookupAddress: Missing docstring
W9208: 84,4:ResolverBase.lookupIPV6Address: Missing docstring
W9208: 88,4:ResolverBase.lookupAddress6: Missing docstring
W9208: 92,4:ResolverBase.lookupMailExchange: Missing docstring


Revision history for this message
Richard Wall (richardw) wrote :

I wrote a very hacky first attempt in lp:~richardw/twistedchecker/inherited-interface-documentation-1132540

I guess it needs refactoring, testing and possibly reworking so that it doesn't actually import the interface classes.

Revision history for this message
Richard Wall (richardw) wrote :

Tom Prince did an initial code review and I have since improved the branch so that it no longer imports any modules.

If someone has time to look at it I'd appreciate another code review. See the review request above.

HawkOwl (hawkowl)
Changed in twistedchecker:
status: New → Confirmed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers