document how to use rcov #719

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
4 participants
Contributor

charlietanksley commented Oct 23, 2012

I'm a little uncertain about this one. I didn't know if it needed associated features or if a README change was sufficient, and I didn't know if all these methods should be documented (or if there were others?), but I thought I needed to start somewhere. Let me know what needs to change and I'm happy to revise. 😺

Closes #479
Closes #551

I just read through the issues and it looks clear to me, putting it in as a feature would mean pulling in rcov to run those features, so README changes makes sense.

correctly document using rcov with rspec rake task
My earlier commit simply had the facts wrong.
Contributor

charlietanksley commented Oct 24, 2012

I realized overnight that I documented this incorrectly! The second commit actually gets the facts right, I think. If this is sufficient, I'm happy to squash these into one commit and force push on my end before you merge.

Owner

myronmarston commented Oct 27, 2012

This looks OK to me, although I don't use RCov and haven't in a long time. @alindeman / @dchelimsky -- can one of you confirm this is correct?

Contributor

alindeman commented Oct 27, 2012

I haven't used rcov in a while either, but reading the other issues suggests the language is correct! I'll merge shortly.

@alindeman alindeman closed this in f442561 Oct 27, 2012

Contributor

alindeman commented Oct 27, 2012

Thanks @charlietanksley 🌈

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