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

Remove coveralls.io integration #1196

Closed
jarifibrahim opened this issue Jan 15, 2020 · 4 comments · Fixed by #1219
Closed

Remove coveralls.io integration #1196

jarifibrahim opened this issue Jan 15, 2020 · 4 comments · Fixed by #1219
Labels
area/tools Issues related to maintenance tools and CLI. exp/beginner Something most people could solve. good first issue These are simple issues that can be picked up by new contributors kind/maintenance Maintenance tasks, such as refactoring, with no impact in features.

Comments

@jarifibrahim
Copy link
Contributor

Coverall.io integration fails frequently which leads to a failed build. We should remove it.

See example build https://travis-ci.org/dgraph-io/badger/jobs/636936733?utm_medium=notification&utm_source=github_status

badger/.travis.yml

Lines 24 to 29 in 5870b7b

before_script:
- go get github.com/mattn/goveralls
script:
- bash contrib/cover.sh $HOME/build coverage.out || travis_terminate 1
- goveralls -service=travis-ci -coverprofile=coverage.out || true
- goveralls -coverprofile=coverage.out -service=travis-ci

@jarifibrahim jarifibrahim added kind/maintenance Maintenance tasks, such as refactoring, with no impact in features. area/tools Issues related to maintenance tools and CLI. exp/beginner Something most people could solve. good first issue These are simple issues that can be picked up by new contributors labels Jan 15, 2020
@janisz
Copy link
Contributor

janisz commented Jan 22, 2020

How about replacing it with https://codecov.io/
Integration is much simpler

after_success:
- bash <(curl -s https://codecov.io/bash)

@vardhanapoorv
Copy link
Contributor

@janisz the code coverage isn't a reliable metric, hence it would be better to remove it.
Feel free to send a PR.

janisz added a commit to janisz/badger that referenced this issue Feb 13, 2020
@janisz
Copy link
Contributor

janisz commented Feb 19, 2020

@vardhanapoorv test are not working :(

@jarifibrahim
Copy link
Contributor Author

Hey @janisz . I am aware of the test failure. There seems to be some configuration issue with the build. We'll fix it soon :)

Thank you for your PR.

jarifibrahim pushed a commit that referenced this issue Mar 4, 2020
jarifibrahim pushed a commit that referenced this issue Mar 12, 2020
Fixes: #1196
(cherry picked from commit 3d95b94)
jarifibrahim pushed a commit that referenced this issue Mar 24, 2020
Fixes: #1196
(cherry picked from commit 3d95b94)
manishrjain pushed a commit to outcaste-io/outserv that referenced this issue Jul 6, 2022
acodereviewersbestfriend52 added a commit to acodereviewersbestfriend52/badger that referenced this issue Aug 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tools Issues related to maintenance tools and CLI. exp/beginner Something most people could solve. good first issue These are simple issues that can be picked up by new contributors kind/maintenance Maintenance tasks, such as refactoring, with no impact in features.
Development

Successfully merging a pull request may close this issue.

3 participants