Ivan Andrus <darthand...@gmail.com> writes: > 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.
Yup. (The latter would of course be better.) But it would be on everyone's local sage install, not on trac or in the release management scripts :) -Keshav -- 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.
