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

adding "details" to tag whitelist #139

Merged
merged 1 commit into from
Sep 3, 2014
Merged

adding "details" to tag whitelist #139

merged 1 commit into from
Sep 3, 2014

Conversation

tansaku
Copy link
Contributor

@tansaku tansaku commented Sep 2, 2014

Apologies if this pull request is to the wrong branch, or in any fashion inappropriate. It's to support this feature request: #138 which comes from my desire to use summary/detail tag combinations in github markdown. I'm assuming that if the summary tag is acceptable, then the detail tag is okay too?

I've run the tests locally and they all pass (for ruby-2.1.2). Hoping that this can get into a new version and then into github markup so that everyone can enjoy details/summary goodness in their Github READMEs :-)

I've tested them against all the supported rubies in your submissions guidelines. All good, although some of those rubies are not the latest security releases:

ree-1.8.7-2011.03 1.9.2-p290 1.9.3-p429 2.0.0-p247

did you want to upgrade your submissions guidelines in that regard?

@bkeepers
Copy link
Contributor

bkeepers commented Sep 3, 2014

Since we already include summary, it make sense to me to add details. 👍

jch added a commit that referenced this pull request Sep 3, 2014
adding "details" to tag whitelist
@jch jch merged commit ea3c830 into gjtorikian:master Sep 3, 2014
@jch
Copy link
Contributor

jch commented Sep 3, 2014

🍻

@tansaku thanks for the pull. @bkeepers 🤘 for the quick review. I'll cut a new release after issue 135.

@tansaku
Copy link
Contributor Author

tansaku commented Sep 17, 2014

awesome! thanks guys

@tansaku
Copy link
Contributor Author

tansaku commented Sep 30, 2014

just out of interest - what's the time frame for this going into a new release of the gem? many thanks in advance

@bkeepers
Copy link
Contributor

It looks like it was already released. I'm hoping to pull the latest release into GitHub this week.

@tansaku
Copy link
Contributor Author

tansaku commented Sep 30, 2014

awesome - thanks for keeping me in the loop :-)

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

Successfully merging this pull request may close these issues.

3 participants