Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/366
This LGTM, all tests pass locally. @huikyole any comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user coveralls commented on the issue:
https://github.com/apache/climate/pull/366
[![Coverage
Status](https://coveralls.io/builds/6778587/badge)](https://coveralls.io/builds/6778587)
Coverage increased (+0.1%) to 83.65% when pulling
Github user agoodm commented on the issue:
https://github.com/apache/climate/pull/366
Done @lewismc.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/366
Nice @agoodm any chance of adding to the Unit test for this in
https://github.com/apache/climate/blob/master/ocw/tests/test_local.py ?
---
If your project is set up for it, you can reply to this
Github user coveralls commented on the issue:
https://github.com/apache/climate/pull/366
[![Coverage
Status](https://coveralls.io/builds/6774637/badge)](https://coveralls.io/builds/6774637)
Coverage remained the same at 83.506% when pulling
Github user OCWJenkins commented on the issue:
https://github.com/apache/climate/pull/366
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this