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

rspec setup #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

rspec setup #10

wants to merge 2 commits into from

Conversation

AgIt0
Copy link

@AgIt0 AgIt0 commented Jan 23, 2014

Set up rspec instead of test unit

group :development do
gem 'rspec-rails'
end
EOF
Copy link
Owner

Choose a reason for hiding this comment

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

Is there a compelling reason to do this here in install.sh instead of in the john_henry:install Rake task that sets up the other gem files? Appending to a file is easy in bash, but when you need to insert at specific points it became hard, which is why that bit of code is all in the rake task.

Copy link
Author

Choose a reason for hiding this comment

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

lol you're completely right... sorry about that. The only thing is you run the install task after the initial commit, so the spec files won't be there, but I guess that's no biggie.

@derwiki
Copy link
Owner

derwiki commented Jan 23, 2014

Question about where this PR has rspec being set up. Thanks for contributing!

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.

2 participants