Thanks for this! I've slightly modified the git-cl instructions, to include "make the CC list lilypond-devel". That's part of the git-cl config stage, so you don't need to remember it for every individual patch you upload.
http://codereview.appspot.com/3581041/diff/1/Documentation/learning/fundamental.itely File Documentation/learning/fundamental.itely (right): http://codereview.appspot.com/3581041/diff/1/Documentation/learning/fundamental.itely#newcode2113 Documentation/learning/fundamental.itely:2113: @warning{The @code{Stem_engraver} and @code{Beam_engraver} attach their So far, we don't have any @warning inside the @seealso section. I'd say that unless you're specifically warning about one of our links (dunno what that might mean... "warning: this link might not work?" :), this should either be higher up in the section, or be a known issue. Since you're talking about removing an engraver (which isn't really standard NR-level stuff), I'd suggest making it a knownissue rather than a warning. http://codereview.appspot.com/3581041/diff/1/Documentation/notation/spacing.itely File Documentation/notation/spacing.itely (right): http://codereview.appspot.com/3581041/diff/1/Documentation/notation/spacing.itely#newcode928 Documentation/notation/spacing.itely:928: @warning{Odd page numbers are always on the right. If you want the On 2010/12/11 00:36:29, Carl wrote:
I think that this should be a known issue, rather than a warning.
Agreed, this is definitely better as a known issue. http://codereview.appspot.com/3581041/diff/1/Documentation/notation/staff.itely File Documentation/notation/staff.itely (right): http://codereview.appspot.com/3581041/diff/1/Documentation/notation/staff.itely#newcode278 Documentation/notation/staff.itely:278: @code{PianoStaff} does not, by default, accept @code{ChordNames}. This should be a known issue, instead of a warn... oh, oops, sorry, it's already a known issue. :) http://codereview.appspot.com/3581041/ _______________________________________________ lilypond-devel mailing list [email protected] http://lists.gnu.org/mailman/listinfo/lilypond-devel
