Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix paragraph #1249

Merged
merged 1 commit into from
May 5, 2018
Merged

fix paragraph #1249

merged 1 commit into from
May 5, 2018

Conversation

UziTech
Copy link
Member

@UziTech UziTech commented May 2, 2018

Marked version: 41a8cad

Description

Contributor

  • Test(s) exist to ensure functionality and minimize regression

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

@Feder1co5oave
Copy link
Contributor

Given some thought I concluded that this change has the benefit of recognizing as paragraphs one-character paragraphs too:

master:

bin/marked --tokens
p
[
  {
    "type": "text",
    "text": "p"
  }
]

pr:

bin/marked --tokens
p
[
  {
    "type": "paragraph",
    "text": "p"
  }
]

I couldn't find any regressions yet

@styfle styfle merged commit 7a9e35e into markedjs:master May 5, 2018
@UziTech UziTech deleted the p-fix branch February 19, 2019 20:15
zhenalexfan pushed a commit to zhenalexfan/MarkdownHan that referenced this pull request Nov 8, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants