-
Notifications
You must be signed in to change notification settings - Fork 391
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
Fix missed rst->md numbered list syntax for m.room.server_acl
#3681
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like it does exactly what it says on the box.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please can we use real numbers and not markdown trickery
Oh, why do you prefer real numbers? I find the advantage of this is that items can be added to the list without having to reflow the numbers. |
It's so that the raw markdown can theoretically be used as a reference without the HTML rendering, and to make review easier down the line when we say "Rule 4 is changed" |
Thank you, I hadn't considered those points. That sounds like reason enough to change it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks :)
This should also get a typos changelog (copy/paste from an existing one)
oh, it has a changelog and I'm blind. |
There was some left-over RST syntax which was causing a numbered list to have broken formatting:
This PR applies the equivalent markdown syntax.
Preview: https://pr3681--matrix-org-previews.netlify.app