2011/1/3 Héctor Rivas Gándara <[email protected]>:
> On Mon, Jan 3, 2011 at 8:58 PM, Daniel Pittman <[email protected]> wrote:
>> On Mon, Jan 3, 2011 at 02:21, Hector Rivas Gandara <[email protected]> wrote:
>>> From: Hector Rivas Gandara <[email protected]>
>>
>> This could do with a better comment about what the change is for.
>
> I can rewrite the comments, but I do not know if you merged the branch.
Even if we don't, having a short explanation of why you changed
something makes it much easier to see what this patch is about and to
give it a good review. Generally, though, we do like to pull your
changes and make sure your credits stay in place. :)
Regards,
Daniel
--
✉ Daniel Pittman <[email protected]>
⌨ [email protected] (XMPP)
☎ +1 503 893 2285
♻ made with 100 percent post-consumer electrons
--
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.
