"to and fro" is correct

[EdwardBetts]

codespell suggests replacing "fro" with "for". Can we have an exception for the phrase "to and fro"?

https://en.wiktionary.org/wiki/to_and_fro

[larsoner]

Sure, feel free to add it

[EdwardBetts]

Thanks @larsoner. I couldn't find the an existing mechanism to specify multi word exceptions like this. Does it exist and I've missed it, or would it need to be added to handle this case?

[larsoner]

Usually people just add the word to the list like:

fro->for, fro

Right @luzpaz ?

[larsoner]

(I don't think there is a multi-word way in particular, I think it's okay to just use this single-word one in the meantime)

[luzpaz]

Actually, you're missing a comma at the end:
fro->for, fro,
Sorry, did you mean something like:
https://github.com/codespell-project/codespell/blob/master/codespell_lib/data/dictionary.txt#L655
fro->for, fro is correct if it's in the context of 'to and fro'
something like that ?

[peternewman]

So I've just sort of hit another of these, preform->perform. Well actually in this case it was a typo, but preform is a valid word too. But there are others like Cristal.

cristal->crystal, cristal,

Essentially we currently have some entries where the misspelling is also listed as a valid correction. There are two trains of thought here I guess, firstly it's nonsense and we should remove the entry, or more considered, that cristal is most likely a misspelling, but in some (rare) circumstances you may really mean it (see also fro, until we support multi-word corrections #255).

What's the general feeling? I've not used interactive mode, but there a "did you really mean?", and in manual/automatic the option to skip/ignore all potential ones seems like it might be sensible. Essentially treat likely typos differently from definite typos.

[larsoner]

I've not used interactive mode, but there a "did you really mean?",

That is more or less what these entries seem to do. I agree we could add a parameter to be more suggestive (default) or less suggestive (if the "error" is in the list of corrections, don't prompt or report)

[peternewman]

I guess one of the things I've always liked about Codespell is the fact the dictionary is curated, rather than a list of valid words, and hence doesn't normally trip up on valid but obscure/technical words. It sort of feels to me that it goes against that ethos when words are added to the dictionary which are valid (although admittedly mostly rarely used). Even if the other variant (i.e. the "typo") is listed, but even more so when it's not.

[larsoner]

We probably need a new argument for this, that (I agree) should disable these by default.

--strict LEVEL, where for now 0 means include all and 1 (default) means exclude such self corrections? That leaves the option open for other sorts of strictness types later.

[larsoner]

If so, any volunteer to implement this?

[peternewman]

Would a bitmask make more sense, in case someone doesn't want one of the future strict levels? I'll have to pass on implementing it for now.

It probably also needs a dictionary test that we don't just have wit->wit, i.e. there is at least one alternative replacement.

[larsoner]

Yes the idea is to use binary values so it would be a bitmask (it's just a trivial, future compatible one for now)