Skip to content

Conversation

@zzak
Copy link
Member

@zzak zzak commented Dec 20, 2013

This pull request is work in progress, feel free to send patches to: https://github.com/zzak/sdoc/tree/rdoc-4

JeanMertz and others added 17 commits December 20, 2013 14:21
These are the bare minimum required changes to make the generator pass without errors. Some
things still not working:

* Github link now showing up
* Code blocks not working correctly (using Markdown)
* Probably more errors
This specification is unnecessary because we already specify the version
dependency in sdoc.gemspec
We were going to use this to implement RDoc::TestCase features in the
MiniTest::Spec::DSL, however TestCase won't work this way and will have
to reconsider another way.
Since we only have a one spec file right now, I'm just going to keep
them all under ./spec
@zzak
Copy link
Member Author

zzak commented Dec 21, 2013

I think I'm happy with this, and would like to merge.

If anyone has any objections or feedback before I merge this, I will try to do a beta release sometime by the end of next week.

We still have more to do after the release as much of the code is untested and fragile.

Copy link
Member

Choose a reason for hiding this comment

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

What do you think about allowing ~> 4.0 ? All the tests are green locally with ruby/rdoc#279

By the way thanks for your work, everything seems fine on Rails' master with your branch! ❤️

Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe we could do ~> 4.0 and < 5.0

I'll work out the version dependency soon

@manuelmeurer
Copy link
Contributor

Great stuff! Let's get it merged!

@manuelmeurer
Copy link
Contributor

I'm using this code in http://ruby-docs.org/ and so far it has been working great! 👍

@zzak zzak merged commit 83066f4 into rails:master Dec 26, 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.

5 participants