-
Notifications
You must be signed in to change notification settings - Fork 5
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
Typo=Yes should not apply just because of an extra space #39
Comments
Is that a universal guideline? Personally I would consider extra spaces to be a kind of Typo, and it also explains why the tokens around a |
The guidelines on typos. I think the function of |
Missing spaces are also typos in a broader sense but not flagged with Typo=Yes (Which word would that be on? Both? What if it is a missing space before or after punctuation? etc.). The convention is to use |
OK, it's not my intuition but I can live with it either way. |
So, looking at this more closely, would you say in this example:
The first token should still carry Typo=Yes, no? Or do we assume that "goeswith" covers the statement "this is broken"? If not, then we can't just tell the validator to disallow Typo on goeswith. |
I would put |
I see what you're saying, but I thought in |
What the docs say:
I guess it leaves ambiguous where the Typo/CorrectForm should be if there is a normal typo too. Maybe I should draft a document spelling out the formal constraints at play and pseudocode for producing a canonical representation without typos/misuse of spaces/repair. |
Forgot to answer: yes, that would be great! Upon thinking about it, I think I'd prefer for typo to be on the first token in |
I believe 4 of these 5 instances are incorrect because the extra space is already accounted for by
goeswith
.The text was updated successfully, but these errors were encountered: