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

switch to github flavored markdown for CMS pages #1613

Merged
merged 1 commit into from Jul 14, 2020
Merged

switch to github flavored markdown for CMS pages #1613

merged 1 commit into from Jul 14, 2020

Conversation

@ewdurbin
Copy link
Member

@ewdurbin ewdurbin commented Jul 14, 2020

More and more becoming defacto which leads to confused users... rather than battle with extensions and the Markdown python module... just use cmarkgfm.

Copy link
Member

@Mariatta Mariatta left a comment

Makes sense for this change. Thanks.

@ewdurbin
Copy link
Member Author

@ewdurbin ewdurbin commented Jul 14, 2020

Thank you for the review @Mariatta!

@ewdurbin ewdurbin merged commit e60fba5 into master Jul 14, 2020
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@ewdurbin ewdurbin deleted the ghfmd branch Jul 14, 2020
RENDERERS = deepcopy(DEFAULT_MARKUP_TYPES)
for i, renderer in enumerate(RENDERERS):
if renderer[0] == 'markdown':
markdown_index = i

RENDERERS[markdown_index] = (
'markdown',
cmarkgfm.github_flavored_markdown_to_html,
'Markdown'
)
Comment on lines +51 to +60

This comment has been minimized.

@di

di Jul 14, 2020
Member

Suggested change
RENDERERS = deepcopy(DEFAULT_MARKUP_TYPES)
for i, renderer in enumerate(RENDERERS):
if renderer[0] == 'markdown':
markdown_index = i
RENDERERS[markdown_index] = (
'markdown',
cmarkgfm.github_flavored_markdown_to_html,
'Markdown'
)
RENDERERS = [
x, cmarkgfm.github_flavored_markdown_to_html if x == 'markdown' else xx, xxx,
for x, xx, xxx in deepcopy(DEFAULT_MARKUP_TYPES)
]

I don't know what to name these but maybe this would be more clear? idk

@di
di approved these changes Jul 14, 2020
Copy link
Member

@di di left a comment

Approved regardless

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.