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

Re: auto merge bug

From
Jeff King <peff@peff.net>
Date
Mar 6, 2013, 09:15 UTC
Message-ID
<20130306091515.GC2018@sigill.intra.peff.net>
In-Reply-To
<CAOFaZ+4oUD7eMvFmtPdca4AYooxW-PCOiPBUb0jjVw4LPBN8+Q@mail.gmail.com>
On Tue, Mar 05, 2013 at 11:13:12PM +0100, David Krmpotic wrote:
Show 12 quoted lines
> Hi guys! Thank you for responses.. I haven't suspected that repos
> created via GitHub windows app would have union set by default :( have
> to ask them about it.. it seems wrong to me… Here are the defaults for
> a windows repo created with GitHub for windows app:
> [...]
> # Custom for Visual Studio
> *.cs     diff=csharp
> *.sln    merge=union
> *.csproj merge=union
> *.vbproj merge=union
> *.fsproj merge=union
> *.dbproj merge=union

Yeah, I think defaulting to merge=union there is questionable. In an ideal world, the GitHub for Windows folks would ship a specialized merge helper for handling VS project files. It can be open-source and distributed separately for people who don't use GitHub, but they can integrate it seamlessly into the GitHub client. So everybody wins.

I see you've already written to GitHub support; thanks. I'll make sure your issue gets routed to the right people, and I'll see if I can convince them to write the specialized tool. :)

-Peff
Previous: David Krmpotic
Message 9 of 9 in “auto merge bug”
  1. David KrmpoticMar 4, 2013
  2. Jeff KingMar 5, 2013
  3. Jeff KingMar 5, 2013
  4. Junio C HamanoMar 5, 2013
  5. Jeff KingMar 5, 2013
  6. Junio C HamanoMar 5, 2013
  7. Andreas EricssonMar 5, 2013
  8. David KrmpoticMar 5, 2013
  9. Jeff KingMar 6, 2013

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.