-
-
Notifications
You must be signed in to change notification settings - Fork 892
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
Feature request: Add a commonmark
option
#129
Comments
Yes. I got so many requests for tables and other non-commonmark features that I decided to switch. I could enable the commonmark option, but I'd have to make sure documentation clearly stated that certain features simply won't work in that mode. |
I understand. And want to clarify that Commonmark spec allows adding any extensions we want (like tables). It's main goal is to make parsing more deterministic by strict syntax. There is even list with proposed extensions. Also github made there flavors upon commonmark spec. So I think it's a good idea to continue using commonmark ;) |
At least we should have ability to pass |
I'd be open for adding the option. I don't want to enable it by default - I think that will lead to more support issues rather than less (though I appreciate a stricter format on a personal level). |
commonmark
option
Not going to argue 👍 But please mention in Readme that |
Also it would be good to point that |
commonmark
optioncommonmark
option
This is now (kind of) possible by using the
|
It seams that
react-markdown
staring from3.0.0
isn't commonmark compliant any more becausecommonmark
option is disabled by default inremark-parse
. It's more breaking than public api changes and there is no notes about it. This was done on purpose?The text was updated successfully, but these errors were encountered: