Ticket #2380 task closed fixed

Opened 6 years ago

Last modified 3 months ago

Make it possible to update the website with new documentation

Reported by: exarkun Owned by: therve
Priority: normal Milestone: totally automated release infrastructure
Component: release management Keywords:
Cc: exarkun Branch: branches/build-docs-2380-2
(diff, github, buildbot, log)
Author: thijs, therve Launchpad Bug:

Description

The lore-processed howto documentation is presented on the website. Updating this documentation is extremely time consuming.

The release process and the website should be organized such that there is a simple command which will cause the new documentation to appear on the website.

Attachments

build-docs-2380.patch Download (5.7 KB) - added by thijs 17 months ago.

Change History

1

Changed 3 years ago by exarkun

#4355 was a duplicate of this.

2

Changed 3 years ago by exarkun

  • component changed from core to release management

3

Changed 2 years ago by <automation>

  • owner radix deleted

Changed 17 months ago by thijs

4

Changed 17 months ago by thijs

  • owner set to thijs
  • status changed from new to assigned
  • milestone set to regular-releases

5

Changed 17 months ago by thijs

  • branch set to branches/build-docs-2380
  • branch_author set to thijs

(In [33530]) Branching to 'build-docs-2380'

6

Changed 17 months ago by thijs

(In [33535]) add build-docs script, refs #2380

7

Changed 17 months ago by thijs

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

I copied the temporary script on [ReleaseProcess#Buildhowtodocumentsforwebsite] and hooked it up to a build-docs script.

$ bin/admin/build-docs
Must specify three arguments: Twisted checkout path, destination path, and template path.
$ bin/admin/build-docs . ./book ../../../../../twisted-website/docs/website-template.tpl 
pnmtops: writing color PostScript...
pnmtops: writing color PostScript...
pnmtops: writing color PostScript...
pnmtops: writing color PostScript...
pnmtops: writing color PostScript...
pnmtops: writing color PostScript...
pnmtops: warning, image too large for page, rescaling to 0.93578
pnmtops: writing color PostScript...
ls -lh book/
total 1.4M
-rwxrwxr-x 1 thijs thijs 1.4M 2012-02-10 18:14 book.pdf

8

Changed 16 months ago by jonathanj

  • owner set to thijs
  • keywords review removed

Thanks for doing this.

My main concern here is the lack of test coverage for the behaviour of BuildDocsScript.buildDocs. I think breaking buildDocs into its smaller components and parameterizing the builder type would make the code both easier to test and read.

9

Changed 3 months ago by therve

  • branch changed from branches/build-docs-2380 to branches/build-docs-2380-2
  • branch_author changed from thijs to thijs, therve

(In [37718]) Branching to 'build-docs-2380-2'

10

Changed 3 months ago by therve

  • keywords review added
  • owner thijs deleted

Alright, there is one test now, plus a couple of other fixes. Please review!

11

Changed 3 months ago by jonathanj

  • keywords review removed
  • owner set to therve

Thanks for coming back to this!

I think the only item is that setupTeXFiles, _setupTeXSections and _setupTeXBook need docstrings.

12

Changed 3 months ago by therve

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

(In [37746]) Merge build-docs-2380-2

Authors: thijs, therve Reviewer: jonathanj Fixes: #2380

Add a script to build documentation for the release.

Note: See TracTickets for help on using tickets.