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

Add writeup about using with Spring to README #341

Merged
merged 1 commit into from
Feb 19, 2015

Conversation

swrobel
Copy link
Contributor

@swrobel swrobel commented Oct 24, 2014

No description provided.

@@ -603,6 +603,11 @@ SimpleCov is built in [Continuous Integration] on 1.9.3, 2.0.0, 2.1.0 and ruby-h

Try [coverband](https://github.com/danmayer/coverband).

## Want to use Spring with SimpleCov?

See [Make Spring & SimpleCov play nice](http://www.stefanwrobel.com/make-spring-simplecov-play-nice)
Copy link
Collaborator

Choose a reason for hiding this comment

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

would you mind basically transplanting your blog post here? (Also perhaps rename the heading to 'Configuring Spring to work with simplecov'). I'm also curious about what the use-case would be for using spring with simplecov?

Create a config/spring.rb file and move your SimpleCov config into it from your spec_helper.rb file. For example:

if ENV['RAILS_ENV'] == 'test'
  require 'simplecov'
  SimpleCov.start
end

@swrobel
Copy link
Contributor Author

swrobel commented Oct 29, 2014

Updated based on @bf4's request

@swrobel
Copy link
Contributor Author

swrobel commented Nov 5, 2014

@bf4 is this acceptable?

@swrobel swrobel closed this Dec 2, 2014
@bf4 bf4 reopened this Jan 29, 2015
colszowka added a commit that referenced this pull request Feb 19, 2015
Add writeup about using with Spring to README
@colszowka colszowka merged commit 071af5f into simplecov-ruby:master Feb 19, 2015
@colszowka
Copy link
Collaborator

@swrobel Thanks, merged!

colszowka added a commit that referenced this pull request Feb 19, 2015
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