-
Notifications
You must be signed in to change notification settings - Fork 12
Open
Description
Some issues I noticed with the sentence case rule:
- The rule fails with leading emojis. For instance, the line
new Notice("✅ Text is good, nothing to change.");is reported by the rule. - As opposed to names like "iOS", "OpenAI" is apparently not in the default list of exception words, resulting in a lot of false positives if you have an AI plugin.
- The rule also reports the line
.setPlaceholder("sk-123456789…"), even though, arguably, the sentence case requirement does not make sense for a placeholder. - The string
'If there is a file "Template.md" at the root of the vault, …'is marked due to the "Template", even though this is correct here due to the use of quotes. new Notice("Cannot assign id to line.")triggers the sentence case rule, even though the sentence is correct. The issue appears to be that "id" should be spelled "ID," probably because that's one of the acronym-exceptions of the rule. While requiring "ID" to be uppercased makes sense, it is confusing that the line just says "sentence case issue".- Similar to 5.,
.setDesc("Separator used when exporting as .csv file.")triggers the rule, since "csv" is not capitalized. But as opposed to 5., ".csv" is lowercased when used as an extension, making the lowercased spelling valid. (While 5. is technically only a wrong description) - Another issue is that parentheses apparently break the rule. For instance, this gets reported, even though they are obviously two sentences:
new Notice("Foobar. (Hello.)");
Zachatoo, saberzero1, claremacrae and ebullient
Metadata
Metadata
Assignees
Labels
No labels