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

process's or process' #2928

Closed
DimitriPapadopoulos opened this issue Jul 8, 2023 · 1 comment · Fixed by #2935
Closed

process's or process' #2928

DimitriPapadopoulos opened this issue Jul 8, 2023 · 1 comment · Fixed by #2935
Labels
dictionary Changes to the dictionary

Comments

@DimitriPapadopoulos
Copy link
Collaborator

DimitriPapadopoulos commented Jul 8, 2023

Later versions of the U.S. Government Publishing Office Style Manual suggest the s is dropped.

2008

Apostrophes and possessives

8.3. The possessive case of a singular or plural noun not ending in s is formed by adding an apostrophe and s. The possessive case of a singular or plural noun ending in s or with an s sound is formed by adding an apostrophe only.
[...]
boss’, bosses’
[...]

2000

Apostrophes and possessives

8.3. All singular nouns form their possessive case by the addition of an apostrophe and an s. Plural nouns ending in s form their plurals by adding only an apostrophe.
[...]
boss’s, bosses’
[...]

Perhaps it's time we drop such fixes, especially those that are wrong any way (files'->file's could be plural):

$ grep -R "s'->" codespell_lib/data/
codespell_lib/data/dictionary.txt:aircrafts'->aircraft's
codespell_lib/data/dictionary.txt:gaus'->Gauss'
codespell_lib/data/dictionary.txt:guas'->Gauss'
codespell_lib/data/dictionary.txt:guass'->Gauss'
codespell_lib/data/dictionary.txt:packges'->packages'
codespell_lib/data/dictionary.txt:thats'->that's
codespell_lib/data/dictionary_code.txt:files'->file's
codespell_lib/data/dictionary_code.txt:process'->process's
codespell_lib/data/dictionary_code.txt:subprocess'->subprocess's
@isaak654
Copy link
Contributor

isaak654 commented Jul 8, 2023

I agree, plus this correction produces unnecessary warnings:
SBIEMSG.js:13: process' ==> process's
Line 13 of SBIEMSG.js:
{ id: 'process', name: tr('Name affected process'), type: 'edit', hint: 'Program.exe when applicable' },

Meanwhile, I'm using --ignore-words-list="process'" as workaround.

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
2 participants