Closed agamm closed 5 years ago
Add more tests so we are at least 50% covered then we will merge. fixes https://github.com/agamm/comeback/issues/3
I'm happy to announce that we have passed the 50% coverage in our tests right now.
@yammesicka do you mind adding tests for comeback\config.py
and comeback\paths.py
please?
I'm happy to announce that we have passed the 50% coverage in our tests right now. @yammesicka do you mind adding tests for
comeback\config.py
andcomeback\paths.py
please?
Sure, it would just take some time 'cause I have the Python course which I have procrastinated for some long time.
I'm happy to announce that we have passed the 50% coverage in our tests right now. @yammesicka do you mind adding tests for
comeback\config.py
andcomeback\paths.py
please?Sure, it would just take some time 'cause I have the Python course which I have procrastinated for some long time.
Yeah we get it, take your time. Just remember that you are the one who can bring the green badge to us :)
(Also another badge for coverage(!))
@yammesicka Ok, I got to 65% cov. I think we can merge this if you accept the changes, so we could start having testing.
Continue here: https://github.com/agamm/comeback/issues/43 We merged this because we want to start having basic tests, even if it isn't complete.
Do not approve yet.