Re: [PATCH v6 6.5/6] fixup! worktree: teach "add" to check out existing branches

2018-04-09 Thread Eric Sunshine
On Mon, Apr 9, 2018 at 3:44 PM, Thomas Gummerer wrote: > On 04/08, Eric Sunshine wrote: >> This is making me wonder if "Checking out branch" is perhaps the wrong >> terminology. What if it said something like this instead: >> >> $ git worktree add ../next >>

Re: [PATCH v6 6.5/6] fixup! worktree: teach "add" to check out existing branches

2018-04-09 Thread Thomas Gummerer
On 04/08, Eric Sunshine wrote: > On Sun, Apr 1, 2018 at 9:11 AM, Thomas Gummerer wrote: > > So while playing with it a bit more I found one case where the new UI > > is not ideal and a bit confusing. Namely when the new check out dwim > > kicks in, but there is already a

Re: [PATCH v6 6.5/6] fixup! worktree: teach "add" to check out existing branches

2018-04-08 Thread Eric Sunshine
On Sun, Apr 1, 2018 at 9:11 AM, Thomas Gummerer wrote: > So while playing with it a bit more I found one case where the new UI > is not ideal and a bit confusing. Namely when the new check out dwim > kicks in, but there is already a file/directory at the path we're > giving

[PATCH v6 6.5/6] fixup! worktree: teach "add" to check out existing branches

2018-04-01 Thread Thomas Gummerer
Signed-off-by: Thomas Gummerer --- So while playing with it a bit more I found one case where the new UI is not ideal and a bit confusing. Namely when the new check out dwim kicks in, but there is already a file/directory at the path we're giving to 'git worktree add'. In