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

Several new suggestions #3621

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

mdeweerd
Copy link
Contributor

Here a several new suggestions gathered over the past few months.

@mdeweerd mdeweerd force-pushed the mdeweerd/spellings/2 branch 3 times, most recently from 4a66370 to 6e666d6 Compare January 22, 2025 11:06
@DimitriPapadopoulos DimitriPapadopoulos added the dictionary Changes to the dictionary label Jan 22, 2025
@mdeweerd mdeweerd force-pushed the mdeweerd/spellings/2 branch 2 times, most recently from 0614191 to 26ad5fb Compare January 22, 2025 14:22
Copy link
Contributor

@TheGiraffe3 TheGiraffe3 left a comment

Choose a reason for hiding this comment

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

One more.

@@ -45145,6 +45157,7 @@ pre-proces->pre-process
pre-procesing->pre-processing
pre-realease->pre-release
pre-registeres->pre-registers
pre-selected->preselected
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not sure about this one. Let those who insist on pre-select use it?

@mdeweerd
Copy link
Contributor Author

I checked the dictionnaries, non proposes pre-select, some find 'preselect':

IMHO the approach is that the dictionnaries are the references.
If a user wants to allow pre-select, then it can be added to the ignored words.

@@ -17789,6 +17794,7 @@ counry->country, county,
counsellin->counselling
counsil->counsel
counsils->counsels
counstructor->constructor
Copy link
Collaborator

Choose a reason for hiding this comment

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

Plural:

Suggested change
counstructor->constructor
counstructor->constructor
counstructors->constructors

@@ -31873,6 +31881,9 @@ impplement->implement
impplementating->implementing
impplementation->implementation
impplemented->implemented
impprove->improve
impproved->improved
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
impproved->improved
impproved->improved
impproves->improves
impproving->improving

@@ -31873,6 +31881,9 @@ impplement->implement
impplementating->implementing
impplementation->implementation
impplemented->implemented
impprove->improve
impproved->improved
impprovement->improvement
Copy link
Collaborator

Choose a reason for hiding this comment

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

Plural:

Suggested change
impprovement->improvement
impprovement->improvement
impprovements->improvements

@@ -60227,7 +60242,8 @@ tryin->trying, try in,
tryinng->trying
trys->tries
tryying->trying
ttests->tests
ttempt->attempt, tempt,
Copy link
Collaborator

Choose a reason for hiding this comment

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

All verbal forms?

Suggested change
ttempt->attempt, tempt,
ttempt->attempt, tempt,
ttempted->attempted, tempted,
ttempting->attempting, tempting,
ttempts->attempts, tempts,

@@ -60227,7 +60242,8 @@ tryin->trying, try in,
tryinng->trying
trys->tries
tryying->trying
ttests->tests
ttempt->attempt, tempt,
ttests->tests, attests,
Copy link
Collaborator

Choose a reason for hiding this comment

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

All verbal forms?

Suggested change
ttests->tests, attests,
ttest->test, attest,
ttested->tested, attested,
ttesting->testing, attesting,
ttests->tests, attests,

@@ -63216,6 +63232,7 @@ warantee->warranty
waranties->warranties
waranty->warranty
wardobe->wardrobe
warehosue->warehouse
Copy link
Collaborator

Choose a reason for hiding this comment

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

Plural:

Suggested change
warehosue->warehouse
warehosue->warehouse
warehosues->warehouses

@DimitriPapadopoulos
Copy link
Collaborator

I checked the dictionnaries, non proposes pre-select, some find 'preselect':

Dictionaries usually lack compound words. However, I agree that since preselect is in the dictionary, pre-select should be avoided. See for example: Should that word have a hyphen? I just fear users might complain: native speakers tend to ignore some grammar rules.

@mdeweerd mdeweerd force-pushed the mdeweerd/spellings/2 branch from 6f856aa to 0fa67bd Compare February 21, 2025 16:24
@mdeweerd mdeweerd force-pushed the mdeweerd/spellings/2 branch from 75e252c to 27f3cf4 Compare February 21, 2025 16:47
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