Closed kai687 closed 3 years ago
It looks like this can be addressed by the following in Google.Heading
extends: capitalization
message: "'%s' should use sentence-style capitalization."
link: 'https://developers.google.com/style/capitalization#capitalization-in-titles-and-headings'
level: warning
scope: heading
match: $sentence
indicators:
- ':'
Thank you so much for finding this @ggiesen and taking the time to let me know. This really helps us out.
@kai687
No problem. I actually found the indicators
option while digging through the Microsoft style guide. I couldn't find any documentation on it at all in the vale docs.
Would you mind reopening the issue until my PR (#12) is accepted? It will be closed automatically when it is, and the fix will eventually make it into a release.
According to Google:
should be a valid heading according to Google style. Currently, this triggers the warnings
Google.Colons
andGoogle.Heading
.Changing the scope for
Google.Colons
fixed that warning, but I have no idea how to change theGoogle.Heading
rule to enforce sentence case, except after colon. I failed to come up with a pattern for the exceptions.