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

Fix the regexes for thematic breaks #826

Closed
wants to merge 2 commits into from
Closed

Conversation

kythyria
Copy link
Contributor

Per the CommonMark spec and GFM, you get a <hr> only if all three characters are the same (Example 26).

This makes that change.

ozziexsh and others added 2 commits November 29, 2016 15:31
Changed sanitize default from true to false in the example.
Commonmark and GFM both agree that -_- is literal characters, not <hr>,
as all three characters must be the same.

Also change the test harness to only look for .text and .html files
since it was choking on vim's swapfiles.

And add some tests to catch what this fixes.
@Feder1co5oave
Copy link
Contributor

+1
This is correct, I'd like to review this, resolve conflicts and reopen it.

@joshbruce
Copy link
Member

Do it up @Feder1co5oave - would love to merge this in - I always thought <hr> was seriously overlooked in defining documents.

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

Successfully merging this pull request may close these issues.

4 participants