Road to 100% coverage #187

Merged
merged 7 commits into from Mar 10, 2014

3 participants

@ArturG

I will add more tests soon...hopefully we will get 100% coverage before next week :)

@coveralls

Coverage Status

Coverage decreased (-17.07%) when pulling 0e3bb4e on ArturG:rspec into b4f74ac on hacketyhack:master.

@PragTob
Hackety Hack member

Sounds good, thank you!

The build errored due to some git error, I restarted it and here's to it passing!

Thanks for your hard work!

@coveralls

Coverage Status

Coverage remained the same when pulling a0d2ed1 on ArturG:rspec into b4f74ac on hacketyhack:master.

@ArturG

This bot never says good things... :)

@coveralls

Coverage Status

Coverage remained the same when pulling d39e127 on ArturG:rspec into b4f74ac on hacketyhack:master.

@ArturG

@PragTob you may merge this PR, because I'm gonna be inactive for a while (spring break, yay!). We got 99.51% coverage, however, as you may know, the 100% itself doesn't guarantee lack of bugs... we still have a lot of work to do.

@PragTob
Hackety Hack member

Hey, thanks so much! I will review this shortly (if not please ping me). Enjoy your spring break! =D

@PragTob PragTob merged commit e1d35c0 into hacketyhack:master Mar 10, 2014

1 check passed

Details default The Travis CI build passed
@PragTob
Hackety Hack member

Thanks heaps!

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