Re: Grammar checking

[Gregory Heytings <gregory@heytings.org>]

[-- Attachment #1: Type: text/plain, Size: 422 bytes --]


>
> We need in Emacs a free grammar checker that really is up to the job, 
> not just any free program.  The result should be Emacs features that we 
> can be proud of.
>

It's clearly not as accurate as non-free grammar checkers, but it does a 
reasonable job.  I don't know if that's enough to be proud of it.  To give 
you a feeling of what it does, I attach the 18 potential grammar errors it 
detects in CONTRIBUTE.

[-- Attachment #2: CONTRIBUTE.txt --]
[-- Type: text/plain, Size: 7419 bytes --]

1.) Line 12, column 1, Rule ID: MISSING_GENITIVE[2] premium: false prio=-1
Message: The genitive (’s or s’) may be missing.
Suggestion: Emacs' development; Emacs's development
...s)Contributing"'.)  ** The Emacs repository  Emacs development uses Git on Savannah for its main repository...
                                                ^^^^^^^^^^^^^^^^^                                             
More info: https://www.ef.edu/english-resources/english-grammar/forming-possessive/

2.) Line 16, column 27, Rule ID: LC_AFTER_PERIOD[1] premium: false
Message: If a new sentence starts here, add a space and start with an uppercase letter.
Suggestion:  Email;  email
...r.name 'Your Name'  git config --global user.email 'your.name@example.com'  git config --global...
                                                ^^^^^                                             

3.) Line 85, column 27, Rule ID: UPPERCASE_SENTENCE_START premium: false
Message: This sentence does not start with an uppercase letter.
Suggestion: Bug
...cs-devel@gnu.org.  ** Issue tracker (a.k.a. "bug tracker")  The Emacs issue tracker at https:...
                                                ^^^                                             
More info: https://languagetool.org/insights/post/spelling-capital-letters/

4.) Line 113, column 44, Rule ID: ALLOW_TO[1] premium: false
Message: Did you mean “reading”? Or maybe you should add a pronoun? In active voice, ‘allow’ + ‘to’ takes an object, usually a pronoun.
Suggestion: reading
...y and takes just one line -- this will allow to read NEWS in Outline mode after hiding the body o...
                                                ^^^^^^^                                             
More info: http://www.ef.com/english-resources/english-grammar/gerund-equals-infinitive/

5.) Line 126, column 1, Rule ID: SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA[1] premium: false
Message: A comma may be missing after the conjunctive/linking adverb ‘Otherwise’.
Suggestion: Otherwise,
... those by others, mark the entry with "+++". Otherwise do not mark it.  If your change requires upd...
                                                ^^^^^^^^^                                             
More info: https://languagetool.org/insights/post/linking-words/

6.) Line 156, column 31, Rule ID: EN_UNPAIRED_BRACKETS premium: false prio=-1
Message: Unpaired symbol: ‘"’ seems to be missing
...mark it in its 'ert-deftest' definition with ":tags '(:expensive-test)".  To run tests on t...
                                                ^                                             
More info: https://languagetool.org/insights/post/punctuation-guide/#what-are-parentheses

7.) Line 156, column 38, Rule ID: EN_UNPAIRED_BRACKETS premium: false prio=-1
Message: Unpaired symbol: ‘'’ seems to be missing
... in its 'ert-deftest' definition with ":tags '(:expensive-test)".  To run tests on the enti...
                                                ^                                             
More info: https://languagetool.org/insights/post/punctuation-guide/#what-are-parentheses

8.) Line 198, column 47, Rule ID: NEEDNT_TO_DO_AND_DONT_NEED_DO[3] premium: false
Message: Did you mean “to separate”?
Suggestion: to separate
... Use this for minor commits that do not need separate ChangeLog   entries, such as changes in etc/...
                                                ^^^^^^^^                                             

9.) Line 233, column 15, Rule ID: CONFUSION_RULE_GNU_KNEW premium: false prio=-20
Message: ‘gnu’ (large African antelope) seems less likely than ‘knew’ (past of ‘know’).
Suggestion: knew
...http: when either will do.  In   particular, gnu.org and fsf.org URLs should start with "http...
                                                ^^^                                             

10.) Line 240, column 53, Rule ID: COMMA_PARENTHESIS_WHITESPACE premium: false
Message: Don’t put a space on both sides of a quote symbol.
Suggestion: " ;  "
...ines of the commit message that start with "; " are omitted   from the generated ChangeLog.  ...
                                                ^^^                                             

11.) Line 343, column 8, Rule ID: PLUS_MINUS[2] premium: false
Message: Replace by the typographical symbol.
Suggestion: ±
...etters, digits, and characters of the   set [-+./_]; - the changes don't include unresolved ...
                                                ^^                                             

12.) Line 347, column 15, Rule ID: EN_A_VS_AN premium: false prio=-1
Message: Use “an” instead of ‘a’ if the following word starts with a vowel sound, e.g. ‘an article’, ‘an hour’.
Suggestion: an
...space characters, and indented   lines where a SPC character is immediately followed by a T...
                                                ^                                             
More info: https://languagetool.org/insights/post/indefinite-articles/

13.) Line 402, column 53, Rule ID: FOR_WHATEVER_REASONS[1] premium: false
Message: This idiom is spelled with the singular form “reason”.
Suggestion: for whatever reason
...anges should not be merged to master at all, for whatever reasons.  These should be marked by including someth...
                                                ^^^^^^^^^^^^^^^^^^^^                                             
More info: https://www.merriam-webster.com/dictionary/for%20whatever%20reason

14.) Line 411, column 45, Rule ID: CONSECUTIVE_SPACES[1] premium: false
Message: It seems like there are too many consecutive spaces here.
Suggestion:  
... group, its own development conventions, etc.  The upstream project's code is periodically m...
                                                ^^                                             

15.) Line 438, column 1, Rule ID: UPPERCASE_SENTENCE_START premium: false
Message: This sentence does not start with an uppercase letter.
Suggestion: Or
...al/html_node/elisp/GNU-Emacs-Internals.html  or run 'info "(elisp)Tips"' or 'info "(elisp)GN...
                                                ^^                                             
More info: https://languagetool.org/insights/post/spelling-capital-letters/

16.) Line 452, column 35, Rule ID: COMMA_PARENTHESIS_WHITESPACE premium: false
Message: Don’t put a space before the full stop.
Suggestion: .
...directory  See all the files in admin/notes/* .  In particular, see admin/notes/newfile, see...
                                                ^^                                             

17.) Line 499, column 22, Rule ID: COMMA_PARENTHESIS_WHITESPACE premium: false
Message: Don’t put a space after the opening parenthesis.
Suggestion: [
...ariables: mode: outline paragraph-separate: "[  \f]*$" coding: utf-8 end: 
                                                ^^                          

18.) Line 499, column 25, Rule ID: COMMA_PARENTHESIS_WHITESPACE premium: false
Message: Don’t put a space before the closing parenthesis.
Suggestion: ]
...ables: mode: outline paragraph-separate: "[  \f]*$" coding: utf-8 end: 
                                                ^^                       
Time: 6302ms for 202 sentences (32.1 sentences/sec)