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

Remove socioeconomic entries #3353

Merged
merged 1 commit into from
Jan 9, 2025

Conversation

isaak654
Copy link
Contributor

@isaak654 isaak654 commented Feb 5, 2024

In American English, the term "socioeconomic" is typically written as a single word. However, in British English, it is also acceptable to use the hyphenated form "socio-economic", so it should not be considered an error.

@isaak654 isaak654 requested a review from peternewman as a code owner February 5, 2024 20:22
@isaak654 isaak654 changed the title Move British word in the correct dictionary Move British word to the correct dictionary Feb 5, 2024
@isaak654 isaak654 force-pushed the patch-1 branch 2 times, most recently from 970c3f7 to e7e26ae Compare February 5, 2024 20:32
@DimitriPapadopoulos DimitriPapadopoulos added the dictionary Changes to the dictionary label Feb 7, 2024
@DimitriPapadopoulos
Copy link
Collaborator

References are missing (links to mainstream curated dictionaries and SCOWLS).

@DimitriPapadopoulos
Copy link
Collaborator

From SCOWLS:

@DimitriPapadopoulos
Copy link
Collaborator

DimitriPapadopoulos commented Feb 8, 2024

But then most mainstream curated dictionaries accept both, with socioeconomic as the suggested primary spelling.

@DimitriPapadopoulos
Copy link
Collaborator

DimitriPapadopoulos commented Feb 8, 2024

Finally, Google Ngram Viewer does show socioeconomic in wider use in US English and socio-economic in slightly wider use in GB English.

@DimitriPapadopoulos
Copy link
Collaborator

This is confusing. Perhaps we should accept both and remove these lines.

fpistm added a commit to fpistm/Arduino_Core_STM32 that referenced this pull request May 27, 2024
socio-economic: see codespell-project/codespell#3353 for

shiftIN is a function name so safe to ignore.

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
fpistm added a commit to stm32duino/Arduino_Core_STM32 that referenced this pull request May 27, 2024
socio-economic: see codespell-project/codespell#3353 for

shiftIN is a function name so safe to ignore.

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
skshetry added a commit to iterative/py-template that referenced this pull request May 29, 2024
yarikoptic added a commit to yarikoptic/contributor_covenant that referenced this pull request Jan 7, 2025
Original motivation is from constantly running into this while detecting
typos with codespell.
FTR, to codespell this "typo" was added within
codespell-project/codespell#3280
and here is "recent" discussion on likely allowing both forms:
codespell-project/codespell#3353
but giving more arguments.

It is true that "socio-economic" is also known and generally considered
correct.  But if someone goes to
https://www.merriam-webster.com/dictionary/socio-economic it does list it as
"socioeconomic" and examples follow the version without the dash.  So it seems
to be the prevalent version, and thus why not to use it instead?

.it and .ro might (or not) want to adopt too?

    ❯ git grep socio-economic  # filtered from non translated, repeated
    content/version/1/4/code-of-conduct.ro.md:statut socio-economic, naționalitate, aspect fizic, rasă, religie, identitate sau orientare sexuală.
    content/version/2/1/code_of_conduct.it.md:socio-economico, nazionalità, aspetto, razza, casta, colore della pelle,
@yarikoptic
Copy link
Contributor

FWIW, one of the most common pain points encountering this is https://github.com/EthicalSource/contributor_covenant so I have proposed

One way or another, it would be nice if codespell and covenant played together nicely. Since covenant already adopted by many projects, the easiest course of action would be for codespell to generally allow for socio-economic and be done.

@DimitriPapadopoulos
Copy link
Collaborator

Agreed. @isaak654 Would you be willing to remove the socio-economic->socioeconomic entry altogether?

@isaak654 isaak654 changed the title Move British word to the correct dictionary Remove socioeconomic entries Jan 8, 2025
@DimitriPapadopoulos DimitriPapadopoulos merged commit 2626491 into codespell-project:main Jan 9, 2025
15 checks passed
@DimitriPapadopoulos
Copy link
Collaborator

Thank you @isaak654 for this PR and your patience.

@isaak654 isaak654 deleted the patch-1 branch January 9, 2025 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dictionary Changes to the dictionary
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants