Hey, I'd be happy with A), but perhaps we could have an option D) where when we upload (or merge) patches into sage, we automatically strip all whitespace in the patch file?
Best. Travis On Saturday, November 24, 2012 9:57:34 AM UTC-8, Volker Braun wrote: > > On Saturday, November 24, 2012 4:41:07 PM UTC, P Purkayastha wrote: > >> Actually, what I meant to say is that the current policy encourages one >> to not put trailing white space in edited lines. The suggestion is to >> make this official. > > > Thats precisely why I wrote the post in the first place. Because we don't > have a clear "official" policy people are getting mixed signals. Look, the > traffic light just switched to blue! What does that even mean? The net > effect is that we are deterring people from reviewing patches without > getting the sage library closer to a yet-to-be formalized goal. > > I'm perfectly fine with version A) but I'm very frustrated with "hmm yes > better not but maybe so and in general on a Tuesday perhaps just make up > your own mind". Saying that you "encourage" something is crap, either be > specific or don't say anything at all. > -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To post to this group, send email to sage-devel@googlegroups.com. To unsubscribe from this group, send email to sage-devel+unsubscr...@googlegroups.com. Visit this group at http://groups.google.com/group/sage-devel?hl=en.
