Skip to content
This repository has been archived by the owner on Mar 25, 2022. It is now read-only.

Add support for basic markdown tables #68

Closed
wants to merge 1 commit into from
Closed

Add support for basic markdown tables #68

wants to merge 1 commit into from

Conversation

mblaettler
Copy link

Add optional (default disabled) option to allow basic markdown tables.
The method added will convert the markdown table to a reStructuredText table on the fly
and passes it to docutils to create a node.

Please provide feedback for my suggested solution.

Add optional (default disabled) option to allow basic markdown tables.
The method added will convert the markdown table to a reStructuredText table on the fly
and passes it to docutils to create a node.
@mblaettler
Copy link
Author

These changes do not work with CommonMark 0.7.3 (PR #63)

@mblaettler mblaettler closed this May 4, 2017
@likaiguo
Copy link

wish this feature be merged as soon

@mblaettler
Copy link
Author

@likaiguo This merge request is deprecated. It stopped working with the CommonMark 0.7.3 release.

I'm currently not working on a solution for the current version of recommonmark.
Sorry

@clayrisser
Copy link
Contributor

@mblaettler, take a look at the following pull request.

It has been suggested to rebuild the recommonmark parser to use the Markdown python module. This parser supports extensions and has a very solid extension for parsing markdown tables.

I have created a pull request to add the Markdown parser.
#138

It not only enables full markdown support using the Markdown python module but also opens the door for more features supported by the Markdown modules various extensions.

I have ensured the pull request is fully backward compatible with the CommonMark parser.

In order to use the Markdown parser this pull request implements, you have to explicitly set the parser property in the recommonmark_config to Markdown.

    app.add_config_value('recommonmark_config', {
        'parser': 'Markdown'
    }, True)

You can see my code at the link below.
https://github.com/codejamninja/recommonmark/tree/codejamninja/python-markdown

@simonsan
Copy link

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants