-
Notifications
You must be signed in to change notification settings - Fork 178
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
misc.but No paragraph should start with a 'But'
: checks starts of new lines which may not all indicate new paragraphs (rst/TeX)
#1376
Comments
misc.but No paragraph should start with a 'But'
: checks starts of sentences too, not just paragraphs (rst/TeX)misc.but No paragraph should start with a 'But'
: checks starts of new lines which may not all indicate new paragraphs (rst/TeX)
This is meant to be addressed in the code, by using a case-sensitive match. If you're getting false positives from it anyway, that indicates a problem with the I'll try to verify this later, although the check has been updated in #1371, which may have inadvertently fixed this issue. I'll get back to you either way. |
Thanks for looking into this @Nytelife26 . Sorry, my issues description wasn't clear. I've updated it now.
does not generate a warning. So this case is fine. However, with a capital "But" at the start of a sentence, but not a new paragraph, it still does:
In formats like rst/markdown/TeX, this isn't at the start of a new paragraph. A new paragraph needs to be preceded by an empty line in these formats:
So, I think the check needs to be on the lines of: |
it was perfectly clear, looking back on it. i was tired when i read your issue and wrote the reply, so my apologies for that. this should be relatively trivial to fix, and i'll add a test for it while i'm at it. |
The warning is meant to be shown when "But" is at the start of paragraphs. However, in its current implementation, it seems to check for "But" at the beginning of each new line in the body of the text. This is incorrect for formats that normalise white spaces and allow semantic line breaks. They can have "But" at the start of a new line within a paragraph (at the start of or within a sentence).
In rst/markdown/TeX, a new paragraph must be preceded by an empty line.
So:
The warning should appear at the second "But", but not at the first.
The text was updated successfully, but these errors were encountered: