Skip to content

Conversation

@daguar
Copy link
Collaborator

@daguar daguar commented Mar 15, 2015

...ting app, closes #27

@daguar
Copy link
Collaborator Author

daguar commented Mar 15, 2015

@migurski — As someone not from the Ruby ecosystem, want to see if this makes sense to you. If so, wanna merge?

@migurski
Copy link
Contributor

Almost! It’s a big improvement. I’d remove the git clone part, it’s simultaneously too general and too specific. Does the part about project-specific versions of Ruby get adequately covered in Ruby.md? Will people definitely have bundle available after getting RVM?

@daguar
Copy link
Collaborator Author

daguar commented Mar 17, 2015

I’d remove the git clone part

Done, on the branch.

Does the part about project-specific versions of Ruby get adequately covered in Ruby.md?

I don't think it was adequately covered in there, but I do think the Ruby howto is the right place for Ruby version stuff, so I've migrated some of the additional RVM context I gave over there, and deleted it from the Rails howto.

Will people definitely have bundle available after getting RVM?

I am 99.9% sure the answer is yes. On RVM's install page ( https://rvm.io/rvm/install ) it has a section on "To install without rubygems-bundler and rvm gems..." that says to provide explicit override arguments to avoid installing bundler. I also followed the Ruby/RVM howto recently on @bsmithgall's comp, and it came with bundler.

@migurski migurski merged commit 7f5a693 into codeforamerica:master Mar 17, 2015
@daguar daguar deleted the revamp-rails-guide-to-cover-local-setup branch March 17, 2015 18:03
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.

Revamp Rails howto to cover local setup rather than starting a new project

2 participants