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.

Sign up for GitHub

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

Jump to bottom

False positive complies->compiles #2324

Closed
wants to merge 1 commit into from
Closed

False positive complies->compiles #2324

wants to merge 1 commit into from

Conversation

polluks
Copy link
Contributor

@polluks polluks commented Mar 31, 2022

form of "comply"

@polluks
False positive
dc71f9e 
form of "comply"
@luzpaz luzpaz requested a review from peternewman April 3, 2022 22:40
@peternewman peternewman linked an issue Apr 4, 2022 that may be closed by this pull request
Wrong replacement of "complies" (v. comply) with "compiles" (v. compile) #2273
Open
@peternewman
Copy link
Collaborator

Hi @polluks ,

Thanks for this, please see the discussion in #2273 , essentially the whole rare dictionary is potential false positives, you can always turn it off if you'd prefer.

@peternewman peternewman changed the title False positive False positive complies->compiles Apr 4, 2022
@polluks
Copy link
Contributor Author

polluks commented Apr 4, 2022

I see.

@polluks polluks closed this Apr 4, 2022
@sed-i
Copy link
Contributor

sed-i commented Aug 17, 2022 •
edited
Loading

Today suddenly started complaining about ./LICENSE:128: complies ==> compiles.
Now need to run with codespell --builtin clear.

@peternewman
Copy link
Collaborator

Now need to run with codespell --builtin clear.

Just ignoring that one word via an ignore list or ignore file would be the non-nuclear option here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@peternewman peternewman Awaiting requested review from peternewman

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Wrong replacement of "complies" (v. comply) with "compiles" (v. compile)
3 participants
@polluks @peternewman @sed-i