Em less than 3 chars

[UziTech]

Marked version: 8489411

Description

• Fixes Specific character combination not rendered correctly #1178

Contributor

• Test(s) exist to ensure functionality and minimize regression (if no tests added, list tests covering this PR)

Committer

In most cases, this should be a different person than the contributor.

• Draft GitHub release notes have been updated.
• CI is green (no forced merge required).
• Merge PR

[UziTech]

/cc @Feder1co5oave and @davisjam to make sure I didn't introduce any security issues

[Feder1co5oave]

Lgtm

[UziTech]

I added some edge cases like:

**1* => <p><em>*1</em></p>

This required adding more capturing parentheses so we could run into another problem like #1013 (comment)

[UziTech]

@joshbruce do we want to risk a breaking change like #1059 or wait for this fix til 0.4.x?

[joshbruce]

@UziTech: Think this is like #1203 and should come along with a 0.4.x release. We've done a lot of work around DevOps and even adding a couple of features. Given the number of people apparently losing Marked, we haven't received that much negative feedback re semver all things considered.

[davisjam]

The changed regexes are safe from REDOS

[Feder1co5oave]

Did you disable CI?

[styfle]

CI is running for other PRs, but I don't see this one: https://travis-ci.org/markedjs/marked/pull_requests

[UziTech]

That's weird, I swear it was there initially. I just rebased and it looks like it came back.

[re-fort]

I'm looking forward to the next release included this fix:laughing:

[styfle]

@re-fort Definitely! We are waiting for a couple more fixes to land before we release version 0.4.0.