Skip to content
This repository has been archived by the owner on May 31, 2019. It is now read-only.

Fixed coveralls configuration #60

Merged
merged 1 commit into from
Aug 21, 2018
Merged

Fixed coveralls configuration #60

merged 1 commit into from
Aug 21, 2018

Conversation

psturc
Copy link
Contributor

@psturc psturc commented Aug 21, 2018

Motivation

https://issues.jboss.org/browse/AEROGEAR-7810

What

Added COVERALLS_REPO_TOKEN

@coveralls
Copy link

coveralls commented Aug 21, 2018

Pull Request Test Coverage Report for Build 503

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+37.02%) to 83.516%

Totals Coverage Status
Change from base Build #446: 37.02%
Covered Lines: 819
Relevant Lines: 948

💛 - Coveralls

@psturc psturc changed the title DO NOT MERGE - test of coveralls Fixed coveralls configuration Aug 21, 2018
@darahayes
Copy link
Contributor

Hey Pavel, I originally had the COVERALLS_REPO_TOKEN there and the integration didn't work. I'm going to approve this and see what happens. We may need to look at it again in the future if the problem arises again.

@psturc
Copy link
Contributor Author

psturc commented Aug 21, 2018

@darahayes ok, in case of problems we can reopen the ticket and investigate more || migrate to codecov.io

@psturc psturc merged commit 1b2e1e3 into master Aug 21, 2018
@psturc psturc deleted the test-coveralls branch August 21, 2018 14:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants