[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Doc: LM: Reformat ly code. (issue1056041)
From: |
percival . music . ca |
Subject: |
Re: Doc: LM: Reformat ly code. (issue1056041) |
Date: |
Fri, 07 May 2010 10:47:14 +0000 |
Looks (mostly) good to me, other than the below 2 items.
http://codereview.appspot.com/1056041/diff/33001/27003
File Documentation/learning/fundamental.itely (right):
http://codereview.appspot.com/1056041/diff/33001/27003#newcode1339
Documentation/learning/fundamental.itely:1339: @c TODO: This extended
example (from here to the end of this node)
I don't want any commits to add TODOs to "finished" code; please move
this to your own personal todo list, or add an issue on the tracker for
it.
http://codereview.appspot.com/1056041/diff/33001/27003#newcode2481
Documentation/learning/fundamental.itely:2481: <<
There is *still* an indentation mistake here. This is the third time
I've pointed it out -- do we disagree on the "two-space indents" rule?
http://codereview.appspot.com/1056041/show
- Re: Doc: LM: Reformat ly code. (issue1056041), (continued)
- Re: Doc: LM: Reformat ly code. (issue1056041), Graham Percival, 2010/05/05
- Re: Doc: LM: Reformat ly code. (issue1056041), Carl Sorensen, 2010/05/05
- Re: Doc: LM: Reformat ly code. (issue1056041), Graham Percival, 2010/05/05
- Re: Doc: LM: Reformat ly code. (issue1056041), Carl Sorensen, 2010/05/05
- Re: Doc: LM: Reformat ly code. (issue1056041), Mark Polesky, 2010/05/06
- Re: Doc: LM: Reformat ly code. (issue1056041), Graham Percival, 2010/05/06
Re: Doc: LM: Reformat ly code. (issue1056041), percival . music . ca, 2010/05/06
Re: Doc: LM: Reformat ly code. (issue1056041), percival . music . ca, 2010/05/06
Re: Doc: LM: Reformat ly code. (issue1056041), tdanielsmusic, 2010/05/06
Re: Doc: LM: Reformat ly code. (issue1056041),
percival . music . ca <=