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

README should mention rvm.el #127

Open
davidw opened this issue Jul 23, 2015 · 6 comments
Open

README should mention rvm.el #127

davidw opened this issue Jul 23, 2015 · 6 comments

Comments

@davidw
Copy link
Contributor

davidw commented Jul 23, 2015

rspec-mode did not work for me before installing rvm.el and setting rspec-use-rvm to t. Mentioning this in the README might help people get up and running a bit faster.

@dgutov
Copy link
Collaborator

dgutov commented Jul 23, 2015

I suppose. Right now, it's only mentioned in the package description (try M-x describe-package).

@davidw
Copy link
Contributor Author

davidw commented Jul 23, 2015

@dgutov and even that's more of "you might consider this", rather than "it won't work without this". For a lot of people, rvm is pretty standard.

@dgutov
Copy link
Collaborator

dgutov commented Nov 13, 2015

For a lot of people, rvm is pretty standard.

All of them should switch to rbenv. :)

Patch welcome, by the way.

@davidw
Copy link
Contributor Author

davidw commented Nov 13, 2015

#132

@pedz
Copy link
Contributor

pedz commented Nov 13, 2015

I use to use rvm and regret it. At least once the update was "delete everything and start over". And I never got them to pull in fixes to their scripts. My list of complaints is long. I even gave money to them at one point.

@davidw
Copy link
Contributor Author

davidw commented Nov 13, 2015

@pedz fine, but this is about making rspec-mode friendlier for those who do use it.

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

No branches or pull requests

3 participants