-
Notifications
You must be signed in to change notification settings - Fork 630
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
tracking: improve test coverage #3713
Comments
Hello! |
Hi Paul! This task is best tackled in parts (one PR per sub-module). Which sub-modules would you like to tackle? |
Ok! I started digging in the csv sub-module so let csv be the first. Also I am intreseted in encoding and json. Which one of these two would you recommend for a beginner? |
Hello! I would like to work on some test coverage improvements. |
hi, I'm working on |
|
I think this tracker is not up-to-date anymore. |
Thanks. I've updated the initial comment so that it doesn't require as much attention. |
Achieving 100% test coverage in this repo isn't critical but would be a nice thing to have. It's also an indicator, of some degree, of the quality of the repo. For anyone wanting to help on this front, you can go to this repo on Codecov and choose an area of the codebase that you can add tests for until said area reaches 100% test coverage.
The text was updated successfully, but these errors were encountered: