Coveralls

[c089]

No description provided.

[c089]

Let's wait for travis to see if it works. Note for self, should also add the badge to the README.md.

Got bored at the hostel. With 💌 from Krakow

[lo1tuma]

This fails with node 0.8 (which has an older npm version).

But maybe we should drop node 0.8 support anyway, I think its npm version doesn’t support peerDependencies which was a problem in #246 (cc: @spikebrehm).

[lo1tuma]

👍 for coveralls, a badge would be nice.

[spikebrehm]

Awesome! Never heard of Coveralls. A little scared to see our initial coverage stats 😮.

You're right, we should explicitly drop support for 0.10 due to peerDependencies.

[c089]

Awesome! Never heard of Coveralls. A little scared to see our initial coverage stats 😮.
Well @lo1tuma already did great work at improving it. Coveralls picked up the PR and reports 77%, but for some reason it shows it as being on the master branch?!

[c089]

Regarding dropping 0.8, ACK but I prefer fixed versions anyway so I went that route and will drop 0.8 in a separate PR.

[c089]

Ah of course the branch only exists on my repo, not rendrjs juts a bit confusing that it shows it as "master" but that's probably because I want to merge to master. I think this one is ready to merge.

[coveralls]

Changes Unknown when pulling 5b277b6 on c089:coveralls into * on rendrjs:master*.

[coveralls]

Changes Unknown when pulling ca9dd6e on c089:coveralls into * on rendrjs:master*.