On Nov 25, 2012, at 1:01 AM, Keshav Kini <keshav.k...@gmail.com> wrote:
> Travis Scrimshaw <tsc...@ucdavis.edu> writes: >> 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? > > This will get nasty once we stop using patch files and switch to git. > Commits should not be invisibly edited after they've been made and > "approved" for some value of "approved". But we can have a commit hook, right? It could just refuse to commit with trailing whitespace. Or maybe it could even offer to fix it for you. -Ivan -- 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.
