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

Setup coverage testing, closes #1058 #1059

Closed
wants to merge 1 commit into from

Conversation

AndreMiras
Copy link
Member

Also enables report to coveralls.io via GitHub Action.
Note the COVERALLS_REPO_TOKEN was setup via:
https://github.com/kivy/buildozer/settings/secrets
And is accessible for repos admin via:
https://coveralls.io/github/kivy/buildozer

Also enables report to coveralls.io via GitHub Action.
Note the `COVERALLS_REPO_TOKEN` was setup via:
https://github.com/kivy/buildozer/settings/secrets
And is accessible for repos admin via:
https://coveralls.io/github/kivy/buildozer
@AndreMiras AndreMiras requested a review from opacam April 10, 2020 13:16
@AndreMiras
Copy link
Member Author

Obviously the coverage is extremely low (20%) and we will try to increase it slowly after this pull request is merged

@AndreMiras
Copy link
Member Author

I'm not sure why it didn't load up COVERALLS_REPO_TOKEN even though it did on my fork and it's setup in the project settings here as well. Will re-run and investigate if it still fails

@AndreMiras
Copy link
Member Author

OK this is because I'm pull requesting from the fork and for security the secrets are not exposed.
I'll create an upstream branch instead.
image

@AndreMiras
Copy link
Member Author

Follow up #1060 branching from upstream

@AndreMiras AndreMiras closed this Apr 10, 2020
@AndreMiras AndreMiras deleted the feature/coverage_testing branch April 10, 2020 14:01
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.

1 participant