Re: git patch comments

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Bruno Wolff III wrote:
> On Mon, Jun 13, 2011 at 10:32:59 -0500,
>   Michael Cronenworth <mike@xxxxxxxxxx> wrote:
>>
>> I double checked the patches and indeed they are formatted as:
>>
>> line 1
>> line 2
>> line 3
>>
>> Testing the following formatting of:
>>
>> line 1
>>
>> line 2
>> line 3
>>
>> results in the expected formatting. Slightly annoying behavior but
>> easily worked around. Thanks.
>
> The lines before the first blank line are the subject. You need to
> have that blank line if you add other comments.

Indeed.  And if line1, line2 and line3 represent different changes,
they properly belong in seperate commits.  Since git lets you commit
locally and it's incredibly fast, the old habits of pushing multiple
disparate changes in one commit is a habit that should go away.

Git even makes it pretty easy to make a small series of changes to
various files, then add them as individual commits using 'git add -p'
and 'git rebase', among other methods.

-- 
Todd        OpenPGP -> KeyID: 0xBEAF0CE3 | URL: www.pobox.com/~tmz/pgp
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
There are a thousand hacking at the branches of evil to one who is
striking at the root.
    -- Henry David Thoreau

Attachment: pgpF5G8XctQ_B.pgp
Description: PGP signature

-- 
users mailing list
users@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe or change subscription options:
https://admin.fedoraproject.org/mailman/listinfo/users
Guidelines: http://fedoraproject.org/wiki/Mailing_list_guidelines
[Index of Archives]     [Older Fedora Users]     [Fedora Announce]     [Fedora Package Announce]     [EPEL Announce]     [EPEL Devel]     [Fedora Magazine]     [Fedora Summer Coding]     [Fedora Laptop]     [Fedora Cloud]     [Fedora Advisory Board]     [Fedora Education]     [Fedora Security]     [Fedora Scitech]     [Fedora Robotics]     [Fedora Infrastructure]     [Fedora Websites]     [Anaconda Devel]     [Fedora Devel Java]     [Fedora Desktop]     [Fedora Fonts]     [Fedora Marketing]     [Fedora Management Tools]     [Fedora Mentors]     [Fedora Package Review]     [Fedora R Devel]     [Fedora PHP Devel]     [Kickstart]     [Fedora Music]     [Fedora Packaging]     [Fedora SELinux]     [Fedora Legal]     [Fedora Kernel]     [Fedora OCaml]     [Coolkey]     [Virtualization Tools]     [ET Management Tools]     [Yum Users]     [Yosemite News]     [Gnome Users]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [Fedora Sparc]     [Libvirt Users]     [Fedora ARM]

  Powered by Linux