Skip to content
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

[2.0] XML Conversion #431

Closed
colinodell opened this issue Apr 12, 2020 · 2 comments · Fixed by #629
Closed

[2.0] XML Conversion #431

colinodell opened this issue Apr 12, 2020 · 2 comments · Fixed by #629
Assignees
Labels
do not close Issue which won't close due to inactivity enhancement New functionality or behavior implemented Change has been implemented
Milestone

Comments

@colinodell
Copy link
Member

colinodell commented Apr 12, 2020

The reference parser supports rendering the AST as XML. I would like to have the same functionality in league/commonmark. This would also be a good test of using the AST for purposes other than rendering it as HTML (converting MD -> XML, HTML -> MD, etc.) However, we may need to implement the other changes in this list including the next one:

Being able to visualize the AST with an XML representation would be especially helpful for developers trying to debug custom extensions they're writing.

@colinodell
Copy link
Member Author

For v2.0, I'd like to implement a simple XmlRenderer annotated with @experimental that serves as more of a debugging tool than a production output format. We can then implement full, proper MD -> XML and XML -> MD conversion in v3.0 alongside #419, since both will require some significant changes that I won't want to delay the v2 release.

@colinodell
Copy link
Member Author

MD -> XML actually seems fairly straightforward, so I'm adding full, non-experimental support for that in v2.0.

XML -> MD will come later.

@close-label close-label bot added the implemented Change has been implemented label Jun 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not close Issue which won't close due to inactivity enhancement New functionality or behavior implemented Change has been implemented
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant