Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix integration with rcov when used via bundler #422

Closed
sundbp opened this Issue Jul 11, 2011 · 9 comments

Comments

Projects
None yet
7 participants

sundbp commented Jul 11, 2011

If rcov is invoked via bundler, as would be pretty standard case, rspec/autorun does not get required. For more details and a proposed fix, see this:
90db090#commitcomment-470474

sundbp commented Jul 12, 2011

As a workaround I'm just requiring rspec/autorun in my spec_helper.rb. Seems to work ok. Any drawbacks to that anyone can think of?

Owner

dchelimsky commented Jul 12, 2011

No drawback to requiring it directly.

ches commented Aug 8, 2011

Would making this a default in the generator serve as a viable alternative to 90db090 then? It seems that's causing a lot of headaches, like #431 which just bit me and took awhile to root out, and a require that's conditional on a blacklist of executables seems rather gross to me.

Owner

dchelimsky commented Aug 8, 2011

@ches - yeah, the black list (or a white list) is clearly fragile. The only problem is that leaving out the require would constitute an intentional backward-incompatible change in a minor release. I'd like to think this through a bit more first, but this might be where we end up.

Contributor

Peeja commented Dec 3, 2012

A cross-reference fairy appears!

There's talk of dropping RCov support in #625.

@y-higuchi y-higuchi referenced this issue in trema/trema Dec 27, 2012

Merged

Fix rcov #229

Owner

JonRowe commented Mar 16, 2013

Given the planned removal of support for RCov (see #447, #625), I feel we should close this at this time.

Owner

samphippen commented Mar 28, 2013

I agree with @JonRowe.

Member

cupakromer commented Jun 16, 2013

👍 for closing

Owner

samphippen commented Jun 16, 2013

Closing due to the fact that we're dropping rcov support in RSpec 3.

@samphippen samphippen closed this Jun 16, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment