search

trautonen / coveralls-maven-plugin

Maven plugin for submitting Java code coverage reports to Coveralls web service.
MIT License
312 stars 123 forks source link

Added support for Wercker #99

Closed SHildebrandt closed 8 years ago

SHildebrandt commented 8 years ago

Right now using this plugin with Wercker is messing up things in Coveralls, since it takes the commit's SHA1 as branch name. The result is that you have hundreds of branches with exactly one commit. You can find an example here (we ignored this problem for quite a while).

It would be great if you could incorporate this fix in a future release (soon? :) ).

Werckers environment variables are described here.

Btw. is there a way to really test this (without having to push it to maven central)? I'm pretty sure that it works, but you never know... ;)

coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.02%) to 97.917% when pulling 91b71d948f7610e54818b939af875269d629ab11 on SHildebrandt:master into 99081442d08c763251c3511ed7002a2931666dc7 on trautonen:master.

trautonen commented 8 years ago

Thanks for this one. Never heard of Wercker before. :+1:

You can test the plugin by adding configuration for sonatype repositories in your pom.xml. Just use 4.3.0-SNAPSHOT as your dependency version.