Hi all, would anyone mind if I commit the attached patch? It's 100% backwards compatible and allows for turning plot directive errors into fatal exceptions easily, so one can make sure that docs either build correctly or not at all. This is useful for having examples be a kind of test suite, in addition to any unit tests a codebase may have. Thanks, f
Fernando Perez wrote: > Hi all, > > would anyone mind if I commit the attached patch? > > It's 100% backwards compatible and allows for turning plot directive > errors into fatal exceptions easily, so one can make sure that docs > either build correctly or not at all. This is useful for having > examples be a kind of test suite, in addition to any unit tests a > codebase may have. > > Thanks, > > f > I'm +1 on this. We can have then have the buildbot doc builder enable this when building the docs. (Which are output at http://matplotlib.sourceforge.net/trunk-docs/ and http://matplotlib.sourceforge.net/trunk-docs/Matplotlib.pdf , for reference .) -Andrew
On Wed, Jan 27, 2010 at 6:06 PM, Andrew Straw <str...@as...> wrote: > I'm +1 on this. We can have then have the buildbot doc builder enable > this when building the docs. (Which are output at > http://matplotlib.sourceforge.net/trunk-docs/ and > http://matplotlib.sourceforge.net/trunk-docs/Matplotlib.pdf , for > reference .) Thanks, with your and John's (off-list) approvals, it's committed. The patch that went in had more docs than what I posted here, but the code is identical. Cheers, f
On Thu, Jan 28, 2010 at 2:51 PM, Fernando Perez <fpe...@gm...> wrote: > Thanks, with your and John's (off-list) approvals, it's committed. > The patch that went in had more docs than what I posted here, but the > code is identical. Oops, somehow my commit had never made it, I just noticed now I was trying to use it from another machine. Fixed now, r8110. Cheers, f