Skip to content

dictionary: offer re-queried as well as required for requeried - #4046

Open
ekanshul wants to merge 1 commit into
codespell-project:mainfrom
ekanshul:fix-2840-requeried
Open

ekanshul wants to merge 1 commit into
codespell-project:mainfrom
ekanshul:fix-2840-requeried

Conversation

@ekanshul

Copy link
Copy Markdown
Contributor

"requeried" is a valid past tense of requery and shows up in source (recordsets, caches, omnicompletion). The default dictionary currently maps it only to "required", so -w rewrites legitimate uses.

Offer both suggestions, matching the approach discussed in the issue:

requeried->re-queried, required,

Fixes #2840

"requeried" is the past tense of requery and appears in source code.
Keep required as a suggestion, but do not treat it as the only fix.

Fixes codespell-project#2840
requency->frequency
requenst->request, requests,
requeried->required
requeried->re-queried, required,

@DimitriPapadopoulos DimitriPapadopoulos Oct 12, 2026 •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

requeried is correct, no need to change it to re-queried which is arguably less correct.

Perhaps just move requeried->required to the rare dictionary — or remove this entry altogether.

@DimitriPapadopoulos DimitriPapadopoulos added the dictionary Changes to the dictionary label Oct 12, 2026
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.

Don't automatically fix "requeried"

2 participants