git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: William Duclot <william.duclot@gmail.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH/RFC] rebase: make resolve message clearer for inexperienced users
Date: Wed, 12 Jul 2017 14:29:23 -0700	[thread overview]
Message-ID: <xmqq4luh1gzw.fsf@gitster.mtv.corp.google.com> (raw)
In-Reply-To: <20170710183101.GA13122@Haydn> (William Duclot's message of "Mon, 10 Jul 2017 20:31:01 +0200")

William Duclot <william.duclot@gmail.com> writes:

>>  - The original said "When you have resolved this problem", without
>>    giving a guidance how to resolve, and without saying what the
>>    problem is.  The updated one says "conflict" to clarify the
>>    "problem", and suggests "git add" as the tool to use after a
>>    manual resolition.  
>> 
>>    Modulo that there are cases where "git rm" is the right tool, the
>>    updated one is strict improvement.
>
> I also wrote "<conflicted_file>" when there could be several. Maybe
> 'mark it as resolved with "git add/rm"' would be a better (and shorter)
> formulation?

Another potential source of confusion is if we are seeing "a"
conflict, or multiple ones.  I'd say it is OK to treat the whole
thing as "a conflict" that Git needs help with by the user editing
multiple files and using multiple "git add" or "git rm".  So "mark
it as resolved with 'git add/rm'" is fine, I would think, but
anything that I say about UI's understandability to new people needs
to be taken with a large grain of salt ;-).

> ... I feel like a lot of git messages could be improved this way
> to offer a UI more welcoming to inexperienced user (which is a
> *broad* segment of users). But I am not aware of the cost of
> translation of this kind of patch: would several patches like this
> one be welcomed?

Surely, as long as I can depend on other reviewers who are more
passionate about end-user experience than I am, I'll take such
patches with their help.

Thanks.

  reply	other threads:[~2017-07-12 21:29 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-09 20:25 [PATCH/RFC] rebase: make resolve message clearer for inexperienced users William Duclot
2017-07-10 16:31 ` Junio C Hamano
2017-07-10 18:31   ` William Duclot
2017-07-12 21:29     ` Junio C Hamano [this message]
2017-07-16 11:39       ` Philip Oakley
2017-07-24  9:51         ` Phillip Wood
2017-07-24 20:53           ` Junio C Hamano
2017-07-26 14:37             ` Phillip Wood

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=xmqq4luh1gzw.fsf@gitster.mtv.corp.google.com \
    --to=gitster@pobox.com \
    --cc=git@vger.kernel.org \
    --cc=william.duclot@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).