search

jdiasn / lidarwind

Wind retrievals for the WindCube lidar
Other
15 stars 4 forks source link

[JOSS Review]: test coverage too low #47

Closed tgoelles closed 1 year ago

tgoelles commented 2 years ago

35% test coverage is too low. You don't need 100% but certainly more than 35%. Please improve the coverage. I have seen that you have it in the makefile.

https://github.com/openjournals/joss-reviews/issues/4852

castelao commented 2 years ago

Indeed. Thanks for pointing that out. We will work on it.

jdiasn commented 2 years ago

Dear @tgoelles, thank you for your comment! We agree with you that the test coverage is low. We will be working to improve it.