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 breaks option description #1381

Merged
merged 1 commit into from
Nov 28, 2018
Merged

fix breaks option description #1381

merged 1 commit into from
Nov 28, 2018

Conversation

UziTech
Copy link
Member

@UziTech UziTech commented Nov 27, 2018

Fix the description for the "breaks" option

Marked version: master

Markdown flavor: Docs

Description

Contributor

  • no tests required for 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

Copy link
Member

@styfle styfle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Funny, that's what I thought it did but when I looked at the docs it seemed pretty specific.

The soft/hard break terminology is confusing 😕

Glad we got some consensus on this and it was simply the docs being wrong 😄

Copy link
Member

@joshbruce joshbruce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With Steve. Thanks!

@styfle styfle merged commit bf973d2 into markedjs:master Nov 28, 2018
@UziTech UziTech deleted the breaks-option branch February 19, 2019 20:22
# 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