Skip to content

upgrade ruby and capybara and fix tests - #21

Open
TheJefe wants to merge 5 commits into
mobomo:masterfrom
TheJefe:jk/upgrade_ruby
Open

upgrade ruby and capybara and fix tests#21
TheJefe wants to merge 5 commits into
mobomo:masterfrom
TheJefe:jk/upgrade_ruby

Conversation

@TheJefe

@TheJefe TheJefe commented Nov 9, 2015

Copy link
Copy Markdown

I started to look at this package, and couldn't use it because the outdated version of ruby can't be installed by rvm on modern versions of OSx.

So this PR

  • updates ruby
  • switches to .ruby-version instead of .rvmrc , because it looks like rvm doesn't want us to do it that any more
  • updated the tests to use be true instead of be_true because be_true has been removed.
  • Also, whereever I encountered white spaces, I cleaned those up.

switch to .ruby-version, which is the recommend way to use rvm.
…ersion

and small fixes to get this running without error
- needed selenium-webdriver in the development dependencies since it is
  used in tests
- there were a bunch of white spaces at the end of some of these test
  files
- open3 needed to be required when we use it
- be_true an be_false is now "be true" and "be false"
Tag files are used by editors to quickly search projects, and should be
generated locally, never need to check this in.

DS_Store is some silly OSx file that annoyingly pops up everywhere
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.

1 participant