git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: co-authoring commits

From
Jakub Narębski <jnareb@gmail.com>
Date
Jun 19, 2015, 18:18 UTC
Message-ID
<55845CFE.4070407@gmail.com>
In-Reply-To
<CAOvwQ4jb-w4+Ah3ZhVE0j1aXLx1=8tRN3Wo98tz+G-wEqLGAcA@mail.gmail.com>
On 2015-06-18 at 23:25, Tuncer Ayaz wrote:
> On Thu, Jun 18, 2015 at 12:52 AM, Theodore Ts'o wrote:
>> On Wed, Jun 17, 2015 at 10:26:32PM +0200, Tuncer Ayaz wrote:
[...]
Show 12 quoted lines
>> One could imagine some frankly, quite rare example where there is a
>> team of people who votes on each commit before it gets sent out and
>> where everyone is equal and there is no hierarchy. In that case,
>> perhaps you could set the from field to a mailing list address. But
>> honestly, how often is that *all* of the authors are completely
>> equal[1]?
> 
> For that case something like patchwork, phabricator, or gerrit seems
> to be the logical tool to use, and should ideally leave a trace of
> approvals and such in the resulting commit message(s). If the patch
> management tool takes care of merging the commit(s), it can be harder
> to misattribute signed-off/reviewed-by/etc, which is a good thing.

Doesn't Gerrit (at least) use trailer-like structured *notes* in the 'reviews' category (i.e. refs/notes/reviews ref) to store information about review process?

Show 6 quoted lines
> You could of course use multiple (everybody makes their own) commits,
> where you risk breaking bisectability and avoid the need for equal
> co-authorship support. In pair programming such intermediate commits
> will quite often be fixups, and when you attempt to squash the fixups
> for bisectability's sake, you may get a desire for co-authorship of
> the resulting commit.

Hmmm... I didn't think about the problem of attributing authorship for squashed commits. Though here multiple 'author' headers, or multiline 'author' header would be a better match than 'coauthor' header (which itself doesn't need, I think, the date filed, or does it?)

[This is sent from Thunderbird news, so it should be all right]
-- 
Jakub Narębski
Previous: Tuncer AyazNext: Tuncer Ayaz
Message 18 of 20 in “co-authoring commits”
  1. Tuncer AyazJun 17, 2015
  2. Junio C HamanoJun 17, 2015
  3. Tuncer AyazJun 17, 2015
  4. Junio C HamanoJun 17, 2015
  5. josh@joshtriplett.orgJun 17, 2015
  6. josh@joshtriplett.orgJun 17, 2015
  7. Junio C HamanoJun 17, 2015
  8. Tuncer AyazJun 17, 2015
  9. josh@joshtriplett.orgJun 17, 2015
  10. Junio C HamanoJun 17, 2015
  11. Jakub NarębskiJun 18, 2015
  12. Jeff KingJun 19, 2015
  13. Jakub NarębskiJun 19, 2015
  14. Theodore Ts'oJun 17, 2015
  15. josh@joshtriplett.orgJun 17, 2015
  16. Jason PyeronJun 18, 2015
  17. Tuncer AyazJun 18, 2015
  18. Jakub NarębskiJun 19, 2015
  19. Tuncer AyazJun 19, 2015
  20. Jakub NarębskiJun 19, 2015

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.