Skip to content
This repository has been archived by the owner on Dec 19, 2022. It is now read-only.

Remove violent language #99

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Remove violent language #99

wants to merge 3 commits into from

Conversation

ahillio
Copy link

@ahillio ahillio commented Feb 24, 2022

Please don't use violent language. Use nonviolent, preferably life-affirming, language instead.

@alerque
Copy link

alerque commented Feb 24, 2022

This is not violent language. This is a generic well known English idiom with a known meaning, in this case a positive one. In fact you can search Google for "define killer" and one of the dictionary definitions that comes up is:

very impressive or effective; excellent.

In fact this is the first definition for the work as used grammatically here. You seem to be conflating the noun "killer" with the adjective form for which the first most common definition is positive:

https://www.merriam-webster.com/dictionary/killer

The replacement suggested here is nonsensical and does not convey the same meaning. There is no reason for this change, in the context of software a "killer feature" is well understood and has no relation to violence.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants