Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #007: Set up code coverage and coveralls #8

Merged
merged 2 commits into from
Sep 7, 2014
Merged

Conversation

oskopek
Copy link
Owner

@oskopek oskopek commented Sep 7, 2014

Adds coveralls support

  • Adds an encrypted COVERALLS_TOKEN to travis.yml and enabled cobertura and coveralls on after_success
  • Adds license information to pom
  • Ignores the MavenModelInspection error on the repoToken ${env.COVERALLS_TOKEN} (it's in the environment only in travis, see travis docs)
  • Removed unused out-commented executions part of cobertura from pom
  • Adds sourceEncoding to pom

* Adds an encrypted COVERALLS_TOKEN to travis.yml and enabled cobertura and coveralls on after_success
* Adds license information to pom
* Ignores the MavenModelInspection error on the repoToken ${env.COVERALLS_TOKEN} (it's in the environment only in travis, see travis docs)
* Removed unused out-commented executions part of cobertura from pom
* It's good practice
* Coveralls plugin fails w/o it
oskopek added a commit that referenced this pull request Sep 7, 2014
Fixes #7: Set up code coverage and coveralls
@oskopek oskopek merged commit 7089273 into master Sep 7, 2014
@oskopek oskopek deleted the issue-007 branch September 7, 2014 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant