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

Jump to bottom

Increase test coverage to 100% #1989

Closed
robert-wagner opened this issue Mar 12, 2019 · 3 comments
Closed

Increase test coverage to 100% #1989

robert-wagner opened this issue Mar 12, 2019 · 3 comments
Assignees
@robert-wagner
Labels
Good first issue 🎓 Perfect for beginners, welcome to OpenMined! Help Wanted 👋

Comments

@robert-wagner
Copy link
Contributor

Currently we have 95% coverage. Ideal that coverage would be 100%
@robert-wagner robert-wagner added Help Wanted 👋 Good first issue 🎓 Perfect for beginners, welcome to OpenMined! labels Mar 12, 2019
@robert-wagner robert-wagner self-assigned this Mar 12, 2019
@Ankk98
Copy link

Ankk98 commented Mar 12, 2019

This issue is assigned or I can work on it?

@robert-wagner
Copy link
Contributor Author

@Ankk98 Feel free to work on it! I will be iterating on it over time but would love any help.

@hereismari hereismari mentioned this issue May 7, 2019
Merged
@Ankit-Dhankhar
Copy link
Contributor

@robert-wagner can we integrate https://codecov.io/ for coverage test during the pull request integrated in travis.ci?
Most of organisation use this for check coverage of pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@robert-wagner robert-wagner

Labels
Good first issue 🎓 Perfect for beginners, welcome to OpenMined! Help Wanted 👋
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@karlhigley @robert-wagner @Ankk98 @Ankit-Dhankhar