On Wed, May 28, 2008 at 11:45 AM, Paul Lathrop <[EMAIL PROTECTED]> wrote: > On Wed, May 28, 2008 at 11:40 AM, Steven Jenkins > <[EMAIL PROTECTED]> wrote: >> Perhaps I'm misunderstanding the rule. My understanding is the rule is: >> >> If you fix whitespace or spelling, *only* do that in a given patch. >> *Don't* mix other fixes in. > > That was my understanding as well. Hence "in the course of normal > development" i.e. I'm making a fix and while I make that fix I do a > M-x indent-region so the code is readable, which fixes a whitespace > problem... Or I fix a typo in a comment I notice while I'm working... > > If I have to isolate those changes from my patches I'm gonna stop > trying to contribute patches, because it seems a nightmare to me to > try to isolate general development from fixing things you notice > during that development.
Unless someone convinces me this is a Really Good Idea (tm). --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en -~----------~----~----~----~------~----~------~--~---
