Re: [PATCH] doc: replace jargon word "impact" with "effect"/"affect"

From: Felipe Contreras <felipe.contreras@gmail.com>
Date: May 12, 2021, 17:32 UTC
Message-ID: <609c112066acd_71bd1208aa@natae.notmuch>
In-Reply-To: <20210512170153.GE8544@kitsune.suse.cz>

Michal Suchánek wrote:

Show 8 quoted lines

> On Wed, May 12, 2021 at 11:47:15AM -0500, Varun Varada wrote:
> > So you're saying you're OK with getting rid of all instances of
> > "impact"? I'm for this, but insofar as I searched the code base, I
> > only found the ones I'm changing in my patch (save for a couple that,
> > as a previous reviewer mentioned, are included from other repos, so I
> > left those).
> 
> Yes, I am not opposed to the change in principle.

Good, so you accept you see nothing wrong with "affect".

> You just failed to provide any valid reason.

*In your opinion*.

In my opinion the problems with the word "impact" have been clearly explained.

Cheers.

-- 
Felipe Contreras

$ 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.