Ticket #5555 enhancement closed fixed

Opened 2 years ago

Last modified 2 years ago

Fix epytext markup issues in twisted.words.xish

Reported by: ClearliteX Owned by: ralphm
Priority: normal Milestone:
Component: words Keywords:
Cc: ralphm, jesstess Branch: branches/xish-epytext-5555
(diff, github, buildbot, log)
Author: ralphm Launchpad Bug:

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

1

  Changed 2 years ago by DefaultCC Plugin

  • cc ralphm added

2

  Changed 2 years ago by jesstess

  • cc jesstess added

3

  Changed 2 years ago by ralphm

  • branch set to branches/xish-epytext-5555
  • branch_author set to ralphm

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

4

follow-up: ↓ 8   Changed 2 years ago by ralphm

  • status changed from new to assigned
  • owner changed from ClearliteX 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).

5

  Changed 2 years ago by ralphm

  • status changed from assigned to new
  • keywords review added
  • owner ralphm deleted

I think this should be ready now. Please review.

6

  Changed 2 years ago by ralphm

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

7

  Changed 2 years ago by ralphm

8

in reply to: ↑ 4 ; follow-up: ↓ 9   Changed 2 years ago by thijs

  • owner set to ralphm
  • keywords review removed

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).

9

in reply to: ↑ 8   Changed 2 years ago by ralphm

  • status changed from new to 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.

10

  Changed 2 years ago by ralphm

  • status changed from assigned to closed
  • resolution set to fixed

(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.