-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Disable LanguageTool for Markdown #3379
Comments
There is people that do use languagetool for Markdown and Asciidoc:
Rather that remove support for LanguateTool for asciidoc and markdown that seems to be useful for some I would recommend to g:ale_linters variable to disable it for these file types. |
Changed back title to disable. FWIW I think most of those user would be much more happy by using a proper wrapper instead, the LanguageTool experience on Markdown is simply bad. |
Add this to your configuration for the desired result:
|
@hsanson it's not personal issue, I know that conf in particular already. The issue is about disabling it by default. As stated already, languagetool is not a tool for parsing Markdown, according to the authors themselves. |
Yes, it is not but this could be considered a breaking change I think. |
@JensHeinrich could be a breaking change for those that consider it more useful than problematic, but also could be a relief for those that consider their markdown is being gratuitously filled with false positives. This was overlooked when added, it was a mistake (I was at the issue thread that added languagetool, and thought it was great). |
So far I only got the "Possible typo: you repeated a whitespace" warning and am quite happy with the languagetool integration otherwise. Maybe we can ignore that rule for markdown by default? |
LanguageTool is not proper for handling Markdown content. The presence of the tool in the command line will cause ALE by default to render Markdown files full of irrelevant warnings, simply because LanguageTool doesn't understand the syntax, it can only proofread plain text.
I don't use AsciiDoc, but it may be in the same boat, or worse.
I recommend that ALE changes this and leave LanguageTool for plain text files only.
To use LanguageTool for Markdown, it's best to employ a wrapper, as attempted at #3376 with TeXtidote.
The text was updated successfully, but these errors were encountered: