Fix markdown syntax for external links#2431
Conversation
✅ Deploy Preview for act-rules ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
| [case folding]: https://www.w3.org/TR/charmod-norm/#dfn-case-folding | ||
| [element]: https://dom.spec.whatwg.org/#element | ||
| [normalization form KD]: https://www.unicode.org/glossary/#normalization_form_kd | ||
| [normalization form kd]: https://www.unicode.org/glossary/#normalization_form_kd |
There was a problem hiding this comment.
Maybe I'm missing something but I'm not sure why you changed this line (and the corresponding line in the spelling yaml file) from the upper case "KD" to the lower case "kd". I think that it should be upper case. The Unicode spec uses upper case. The one usage site, on line 19 of this file, uses upper case.
There was a problem hiding this comment.
Never mind. I looked into it for a minute and I found that no other rule uses upper case in one of these "definition IDs", or whatever we're calling them. So I assume there's a reason.
|
My bad. Changes made by some local extensions have been accidentally committed here. Will restore with just the markdown fixes |
Correct parentheses for markdown link urls.
Need for Call for Review: none (this is chore more than anything else)
Pull Request Etiquette
When creating PR:
developbranch (left side).After creating PR:
Rule,DefinitionorChore.When merging a PR:
How to Review And Approve