Skip to content

Conversation

@jdx
Copy link

@jdx jdx commented Dec 15, 2012

ri isn't used by most developers and this is way too early in the program to start teaching it.

@kerrizor
Copy link
Contributor

Travis build failure, without too much detail (of course 😀 )

In general I'm in favor of removing ri/rdoc stuff. Not that it isn't useful, but I think for the purposes of a day of introduction and overview, the more focused we can be on the core concepts that we want to show off "look how simple and easy this is!" the better. The feedback I've gotten from students after InstallFest runs about 3:1 as "Wow, that was a lot of magic things we had to do.. this Ruby and Rails stuff is complicated."

I would prefer to keep the irb stuff. When I'm teaching or TA'ing an advanced group, I find myself using it to show them an answer to their question about how Ruby works, so from that perspective its really useful tool.

@tjgrathwell tjgrathwell merged commit 6309821 into railsbridge:master Dec 16, 2012
@tjgrathwell
Copy link
Member

@kerrizor : irb is included on the previous page ('ruby language') in just as much detail as it was on this 'tools' page. So I think deleting the whole page is fine.

I'm merging this PR. The "get rid of ri" debate comes up every workshop and I don't think ri has enough fans to merit leaving it in the curriculum proper. Individual instructors are of course encouraged to teach ri if they love it so much.

@jdx
Copy link
Author

jdx commented Dec 17, 2012

not sure why the build failed on this one, looked like a travis issue since it passed after it was merged

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