From: Junio C Hamano Date: Mon, 03 May 2021 05:21:12 GMT Subject: Re: [PATCH v4 2/2] merge: fix swapped "up to date" message components Message-ID: In-Reply-To: <20210502051423.48123-3-sunshine@sunshineco.com> Eric Sunshine writes: > + if (verbosity >= 0) { > + if (squash) > + puts(_("Already up to date. (nothing to squash)")); The original scripted Porcelain may have said so, but the placement of full-stop in the above feels a bit strange. Should we rephrase it to Already up to date (nothing to squash). as we are fixing the phrasing now?