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

KISS: be a thinner layer over the Discount library #49

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

sunaku
Copy link

@sunaku sunaku commented Jul 13, 2011

These commits make RDiscount a thinner layer over the Discount library. They introduce the following backwards-incompatible changes:

  • SmartyPants is now enabled by default, because that's the default behavior of the Discount library.
  • Many attribute accessors in the RDiscount class were replaced by a single RDiscount#flags accessor which just contains the raw integer bit flags that are passed directly to the underlying Discount library.
  • Integer bit masks in the Discount library are exported (using the same names) as constants in the RDiscount class.
  • An environment variable RDISCOUNT_EXTENSIONS allows people to choose the set of RDiscount extensions and Discount flags that they want to enable at the command-line.

Cheers.

This is necessary to enable the MKD_NOPANTS flag in the underlying
Discount library; otherwise the Markdown test suite will fail.
@davidfstr
Copy link
Owner

Sounds like a useful design change. I will consider this for RDiscount 3.x.

@sunaku
Copy link
Author

sunaku commented Nov 1, 2013

Awesome! I'm looking forward to having this merged into RDiscount 3.x. 😺

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

Successfully merging this pull request may close these issues.

None yet

2 participants