[MRG + 2] CI Report likely-affected documentation files

[jnothman]

In an attempt to make the changed documentation from a PR still more accessible as rendered, this reports a list of doc pages that were most likely affected by the PR. It is reported in the build logs and at a page http://scikit-learn.org/circle?{BUILD_ID}/_changed.html.

Affected files are presumed to be:

• doc/...rst -> .html
• examples/...py -> auto_*.html
• generated API ref that link as source to files modified by the PR

This last may under-generate in cases of class inheritance (not sure?).

It may over-generate where:

• multiple classes or functions are defined in the same file
• the code but not docstring was modified

this over-generation could be fixed by more specific tracking of which source lines correspond to docstrings for which classes/functions... but probably later.

[jnothman]

I hope to see results at http://scikit-learn.org/circle?7364/_changed.html

[TomDLT]

It seems to work, well done!
Where does The following documentation files may have been changed by ... is supposed to appear?

[jnothman]

That currently only appears in the build log. It's easy enough to copy to the HTML

…

On 25 November 2016 at 21:38, Tom Dupré la Tour ***@***.***> wrote: It seems to work, well done! Where does The following documentation files may have been changed by ... is supposed to appear? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#7938 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAEz62bCaF01-hBhx1OdSnTOb4oLiMa3ks5rBrqOgaJpZM4K8DQv> .

[amueller]

looks neat. I don't like adding more shell scripts though ... +.5

[jnothman]

You'd accept it if the first line of the script was '#!/usr/bin/env python'?!

…

On 1 December 2016 at 08:56, Andreas Mueller ***@***.***> wrote: looks neat. I don't like adding more shell scripts though ... +.5 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#7938 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAEz6_4shhXsEc7R9SBv97wCLVJosSvCks5rDfEHgaJpZM4K8DQv> .

[amueller]

I wouild ;)

[jnothman]

ha! well... I'm not rewriting it.

…

On 2 December 2016 at 08:31, Andreas Mueller ***@***.***> wrote: I wouild ;) — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#7938 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAEz69REKmbOV1j6P9q2XcqjmitH43OCks5rDzy0gaJpZM4K8DQv> .

[jnothman]

working with git commands is much easier in bash

…

On 2 December 2016 at 14:07, Joel Nothman ***@***.***> wrote: ha! well... I'm not rewriting it. On 2 December 2016 at 08:31, Andreas Mueller ***@***.***> wrote: > I wouild ;) > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <#7938 (comment)>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/AAEz69REKmbOV1j6P9q2XcqjmitH43OCks5rDzy0gaJpZM4K8DQv> > . >

[tguillemot]

That will simplify the reviews :). Thanks.
+1 after removing the foo obviously !!!

[jnothman]

foo removed.