Opened 10 years ago

Closed 10 years ago

#5555 enhancement closed fixed (fixed)

Fix epytext markup issues in twisted.words.xish

Reported by: Liz Owned by: Ralph Meijer
Priority: normal Milestone:
Component: words Keywords:
Cc: Ralph Meijer, jesstess Branch: branches/xish-epytext-5555
branch-diff, diff-cov, branch-cov, buildbot
Author: ralphm

Description

We have a ton of invalid epytext that produces warnings like shown in ​http://pastebin.com/Hd7F0Z0G when we generate the docs.

Change History (10)

comment:1 Changed 10 years ago by DefaultCC Plugin

Cc: Ralph Meijer added

comment:2 Changed 10 years ago by jesstess

Cc: jesstess added

comment:3 Changed 10 years ago by Ralph Meijer

Author: → ralphm
Branch: → branches/xish-epytext-5555

(In [33867]) Branching to 'xish-epytext-5555'

comment:4 Changed 10 years ago by Ralph Meijer

Owner: changed from Liz to Ralph Meijer
Status: new → assigned

Apologies for taking over this ticket. I wasn't aware that it was meant as a way to give our new contributors some easy tickets.

I removed the references to build-in types like str, unicode, etc., and to external modules.

I left the other warnings for what they are. Apparently, there are quite some issues with the object resolution within Pydoctor itself:

  • Links to (local and external) module attributes are not resolved.
  • In some cases, two modules documenting things that are in the other module, don't get resolved.
  • Local class attributes sometimes don't resolve (like logTraffic in StreamManager).

comment:5 Changed 10 years ago by Ralph Meijer

Keywords: review added
Owner: Ralph Meijer deleted
Status: assigned → new

I think this should be ready now. Please review.

comment:6 Changed 10 years ago by Ralph Meijer

References to related Pydoctor issues: ​lp:685376 and ​lp:957014.

comment:7 Changed 10 years ago by Ralph Meijer

And ​lp:712919.

comment:8 in reply to:  4 ; Changed 10 years ago by Thijs Triemstra

Keywords: review removed
Owner: set to Ralph Meijer

Replying to ralphm:

Apologies for taking over this ticket. I wasn't aware that it was meant as a way to give our new contributors some easy tickets.

I removed the references to build-in types like str, unicode, etc., and to external modules.

I left the other warnings for what they are. Apparently, there are quite some issues with the object resolution within Pydoctor itself:

  • Links to (local and external) module attributes are not resolved.
  • In some cases, two modules documenting things that are in the other module, don't get resolved.
  • Local class attributes sometimes don't resolve (like logTraffic in StreamManager).

Thanks for fixing those. Could you also fix the other coding standard stuff like putting docstrings on 3 lines (without the current prefixing whitespace): 

"""
Generic Streaming XML protocol handler.

The news file can be an empty .misc file, I doubt users will care we fixed an api doc issue (cause there are many).

comment:9 in reply to:  8 Changed 10 years ago by Ralph Meijer

Status: new → assigned

Replying to thijs:

.. Could you also fix the other coding standard stuff ..

It is better to do these changes as part of another ticket, as exarkun rightly mentioned in #twisted.

The news file can be an empty .misc file, I doubt users will care we fixed an api doc issue (cause there are many).

Fixed.

comment:10 Changed 10 years ago by Ralph Meijer

Resolution: → fixed
Status: assigned → closed

(In [33882]) Merge xish-epytext-5555: Fix epytext warnings in xish and jabber modules.

Author: ralphm Reviewer: thijs Fixes: #5555

Note: See TracTickets for help on using tickets.