Ticket #4566 enhancement closed fixed

Opened 3 years ago

Last modified 3 years ago

Improve lore2sphinx buildbot results for `projects/conch`

Reported by: khorn Owned by: Screwtape
Priority: normal Milestone: Lore to Sphinx
Component: core Keywords: documentation
Cc: Branch: branches/improve-lore2sphinx-output-for-projects-conch-4566
Author: screwtape Launchpad Bug:

Description

Make the lore2sphinx buildbot results for projects/conch look as good as possible by modifying the lore sources.

Things to look for:

  1. nested inline markup should be removed
  2. whitespace should be added or removed as needed to make ReST markup render properly
  3. anything else that you can improve by modifying the Lore sources

Note that you probably want to run lore2sphinx on a local copy for development purposes.

You can get lore2sphinx from:  http://bitbucket.org/khorn/lore2sphinx

Attachments

lore-conch-cleanups.diff Download (5.4 KB) - added by Screwtape 3 years ago.
A patch that cleans up the Sphinx rendering of the Conch docs.

Change History

1

Changed 3 years ago by Screwtape

  • status changed from new to assigned
  • owner changed from glyph to Screwtape

Changed 3 years ago by Screwtape

A patch that cleans up the Sphinx rendering of the Conch docs.

2

Changed 3 years ago by khorn

The lore2sphinx buildbot results can be found at:

 http://buildbot.twistedmatrix.com/builds/sphinx-html-15615/

(or at different directories for different revisions...the containing directory is browseable)

3

Changed 3 years ago by Screwtape

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

Changes in this patch:

  • Added trailing spaces to a few lines so that the element on the next line would be correctly rendered, not rendered as raw ReST markup.
  • Fixed some links to point to the correct places.
  • The link to telnet_echo.tac now shows its description.
  • Added descriptions for sshsimpleclient.py and sshsimpleserver.py.
  • Filenames and descriptions on the examples page are separated by hyphens, as they are in the Twisted Core examples page

4

Changed 3 years ago by Screwtape

Oh, and things that aren't fixed: spaces between inline markup and end-of-sentence punctuation.

5

Changed 3 years ago by exarkun

  • owner set to Screwtape
  • keywords review removed

Looks fine, although it's missing a news fragment. One thing I noticed while reviewing this, though, is that code listings no longer have line numbers. Could you file a ticket for that?

6

Changed 3 years ago by Screwtape

One thing I noticed while reviewing this, though, is that code listings no longer have line numbers.

Filed as #4586.

7

Changed 3 years ago by screwtape

  • branch set to branches/improve-lore2sphinx-output-for-projects-conch-4566
  • branch_author set to screwtape

(In [29822]) Create branch improve-lore2sphinx-output-for-projects-conch-4566

8

Changed 3 years ago by screwtape

(In [29823]) Apply lore-conch-cleanups.diff

refs #4566

9

Changed 3 years ago by screwtape

(In [29824]) Address exarkun's code-review comments.

refs #4566

10

Changed 3 years ago by Screwtape

  • keywords review added
  • owner Screwtape deleted

Sphinx output  here.

11

Changed 3 years ago by exarkun

  • keywords review removed
  • owner set to Screwtape

Looks good. Please merge.

12

Changed 3 years ago by screwtape

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

(In [29850]) Merge improve-lore2sphinx-output-for-projects-conch-4566: Tidy conch docs for lore2sphinx conversion.

Author: screwtape Reviewers: exarkun Fixes: #4566

  • Added trailing spaces to a few lines so that the element on the next line would be correctly rendered, not rendered as raw ReST markup.
  • Fixed some links to point to the correct places.
  • The link to telnet_echo.tac now shows its description.
  • Added descriptions for sshsimpleclient.py and sshsimpleserver.py.
  • Filenames and descriptions on the examples page are separated by hyphens, as they are in the Twisted Core examples page

13

Changed 3 years ago by khorn

  • milestone set to Lore to Sphinx
Note: See TracTickets for help on using tickets.