-
Notifications
You must be signed in to change notification settings - Fork 197
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
adding inter-doc links and more explanation of tokens #22
Conversation
@choldgraf you missed that LineComments are an extension |
Also AutoLinks are not an extension |
9c23df2
to
77530fe
Compare
Nice - good catch. Fixed both in the latest push Something I just realized is that this PR will remove some information from the docs, which is the order of precedence for the extended syntax and the commonmark syntax. Can you think of a good place to insert that information back in here? I do think it's more valuable to explicitly highlight which syntax is new vs. standard... |
(depending on how you wanna resolve my last question, this PR is ready to go from me, but if you'd like me to add back in "precedence" information somehow (maybe a link in the code for where precedence is defined?) I'm happy to do so |
I’ll do this in a bit, just adding a FrontMatter token 😄 |
77530fe
to
0af0a30
Compare
@chrisjsewell I just rebased on |
0af0a30
to
8b174fe
Compare
Just added a quick commit here, because I'm too lazy to do my own PR |
heh sounds good - ready to merge if tests are happy? If so then I will squash and merge because I'm not an animal. |
@chrisjsewell btw - how do you feel about just only allowing squash merge for a bit? |
Yeh sounds good to me 👍 |
No description provided.