Re: describing regression checking more explicitely (issue4621041)

[lemniskata . bernoullego]

As you wish :)


http://codereview.appspot.com/4621041/diff/5001/Documentation/contributor/regressions.itexi
File Documentation/contributor/regressions.itexi (right):

http://codereview.appspot.com/4621041/diff/5001/Documentation/contributor/regressions.itexi#newcode201
Documentation/contributor/regressions.itexi:201: @warning{
On 2011/06/15 08:21:32, Graham Percival wrote:
> On 2011/06/15 08:14:54, Janek Warchol wrote:
> > Maybe it would be better placed at the bottom or over point 5?
> > Should this be an Advanced note?

> move it to an advanced note at the bottom of the page.  It's certainly
not
> important enough to be a warning; we want to "save" those for truly
important
> stuff.

Done.

http://codereview.appspot.com/4621041/diff/5001/Documentation/contributor/regressions.itexi#newcode207
Documentation/contributor/regressions.itexi:207: @code{make} and
@code{make check} it (without doing @code{make test-baseline}
On 2011/06/15 08:21:32, Graham Percival wrote:
> remove the parenthesis, just end with:
> @code{make} and @code{make check} it without doing @code{make
test-baseline}
> again.

Done.

http://codereview.appspot.com/4621041/diff/5001/Documentation/contributor/regressions.itexi#newcode224
Documentation/contributor/regressions.itexi:224: available at:
On 2011/06/15 08:21:32, Graham Percival wrote:
> available (relative to the current build/ directory) at:

Done.

http://codereview.appspot.com/4621041/diff/5001/Documentation/contributor/regressions.itexi#newcode227
Documentation/contributor/regressions.itexi:227:
lilypond-git/build/out/test-results/index.html
On 2011/06/15 08:21:32, Graham Percival wrote:
> second thought: I think I prefer:
> out/test-results/index.html

> in order to not confuse people, see above comment.

Done.

http://codereview.appspot.com/4621041/