-
-
Notifications
You must be signed in to change notification settings - Fork 382
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
Let users set the common marker extensions #279
Let users set the common marker extensions #279
Conversation
lib/html/pipeline/markdown_filter.rb
Outdated
@@ -12,6 +12,8 @@ class Pipeline | |||
# | |||
# Context options: | |||
# :gfm => false Disable GFM line-end processing | |||
# :commonmarker_extensions => [ ] Common marker extensions to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the [ ]
doing here? I think this is meant to suggest the default, which is instead written on the next line.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Common marker
=> CommonMarker
assert_equal(doc, iframe_escaped) | ||
end | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One too many blank lines.
Wow, the failures sure are extensive. I'm guessing a recent merge updated |
➡️ #280 |
Can you pull in |
2fc7d41
to
2cb3b0d
Compare
Changes: - Expose the common marker extensions to the context. - Set a default list of ext. - Update docs.
2cb3b0d
to
dc0f98a
Compare
@kivikakk @gjtorikian This should be ready for a re-review. |
Looks good to me! |
Released as 2.7.0. |
Changes: