-
Notifications
You must be signed in to change notification settings - Fork 8
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
Report coverage only on default task #1322
Conversation
ae29ca5
to
0d14646
Compare
@@ -34,6 +34,7 @@ def gemsets | |||
end | |||
|
|||
def use_simplecov | |||
return unless ENV['COVERAGE'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't the caller of use_simplecov
decide whether simplecov is supposed to be used? If not, we should rename the method.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, you're right.
Please rebase staging. |
A simple run of `rspec spec/something_spec.rb:123` spends a lot of time on generating the coverage report. This disables the coverage report by default, except for the default rake task.
0d14646
to
e937ecc
Compare
Done. |
👍 |
…_task Report coverage only on default task
A simple run of
rspec spec/something_spec.rb:123
spends a lot of time ongenerating the coverage report.
This disables the coverage report, except for the default rake task.
If a coverage report is desired even for other tasks, just set the environment variable
COVERAGE
, e.g.: