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

Show summary and description; always Markdown #503

Closed
wants to merge 1 commit into from

Conversation

rking
Copy link

@rking rking commented Jan 2, 2013

As a new gem author, I was pretty puzzled by the contrast between:

Additionally, it seems like many people try to put line breaks in these fields, but the site just puts them all in a <p> unless the text happens to match the very specific regex: /^==/

So this patch should rectify both. I hope you like it, but if not, let's talk about the options. Thanks!

@rking
Copy link
Author

rking commented Jan 2, 2013

Related To

#377 and #395, which do not seem to be working at the moment: https://rubygems.org/gems/minitest

Build Failure

Has been going on for a few commits: https://travis-ci.org/rubygems/rubygems.org/builds

I can debug that later, if you want. But this patch has tests and they pass.

Legacy Issue

Some gem authors are in the habit of redundantly stating the summary in the description, probably due to the display logic of the existing rubygems.org

If you guys want, we could description.gsub summary, '' It's a patch solution, but it could smooth the adoption of this new look.

@qrush
Copy link
Member

qrush commented Jan 2, 2013

I don't want to encourage any use of markdown inside of the gemspec. Just simple plaintext is supported, and then your README on github, docs, etc can be better formatted.

@qrush qrush closed this Jan 2, 2013
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.

None yet

2 participants