Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Edits to the English word list #6442

Merged
merged 1 commit into from
Jun 1, 2022

Conversation

sts10
Copy link
Contributor

@sts10 sts10 commented May 10, 2022

Following the lead of commit 4030aec73cf5d0aecdaeb0aaea09834a3eb465da, I spotted a few more awkward words in the English word list. In this PR, I have replaced them one-for-one with words that I deemed a bit more unobjectionable.

The number of words remains the same (7,603), as well as the range of of word lengths (3-character minimum, 10-character maximum).

Status

Ready for review

Description of Changes

Changes proposed in this pull request:

  • Replace a handful of awkward words in the English word list with more suitable replacements.

Testing

  1. Review new words for appropriateness
  2. Verify that 7,603 unique words are on the new list.

Choose one of the following:

  • I have opened a PR in the docs repo for these changes, or will do so later
  • I would appreciate help with the documentation
  • These changes do not require documentation

Choose one of the following:

  • I have performed a diff review and pasted the contents to the packaging wiki
  • I would like someone else to do the diff review

@sts10 sts10 requested a review from a team as a code owner May 10, 2022 19:44
Copy link
Member

@eloquence eloquence left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good removals and replacements, thanks @sts10!

@eloquence eloquence merged commit 785ea3e into freedomofpress:develop Jun 1, 2022
@sts10 sts10 deleted the wordlist_edits branch June 1, 2022 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants