[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: CG: add information about Regtest Checking Project (issue 5669047)
From: |
graham |
Subject: |
Re: CG: add information about Regtest Checking Project (issue 5669047) |
Date: |
Tue, 14 Feb 2012 18:33:06 +0000 |
http://codereview.appspot.com/5669047/diff/1/Documentation/contributor/regressions.itexi
File Documentation/contributor/regressions.itexi (right):
http://codereview.appspot.com/5669047/diff/1/Documentation/contributor/regressions.itexi#newcode511
Documentation/contributor/regressions.itexi:511: it shouldn't check
everything at once. For example it's a bad idea to test
I disagree slightly here: most regtests should only do a single thing,
but a few (deliberately) test multiple ones. For example, there's some
piece by IIRC Schubert which uses every type of grob (or something like
that; I just remember being told off by Jan for deleting that regtest so
I had to rescue it from git history).
Ideally those regtests would be clearly marked as separate from the
others, but since that hasn't been done, I don't think we should ask
users to complain about large regtests.
http://codereview.appspot.com/5669047/
- CG: add information about Regtest Checking Project (issue 5669047), janek . lilypond, 2012/02/14
- Re: CG: add information about Regtest Checking Project (issue 5669047),
graham <=
- Re: CG: add information about Regtest Checking Project (issue 5669047), janek . lilypond, 2012/02/14
- Re: CG: add information about Regtest Checking Project (issue 5669047), graham, 2012/02/17
- Re: CG: add information about Regtest Checking Project (issue 5669047), janek . lilypond, 2012/02/22
- Re: CG: add information about Regtest Checking Project (issue 5669047), janek . lilypond, 2012/02/25
- Re: CG: add information about Regtest Checking Project (issue 5669047), janek . lilypond, 2012/02/26