search

thedataincubator / forecaster

3 stars 1 forks source link

Index page #16

Closed craw-daddy closed 6 years ago

craw-daddy commented 6 years ago

Rebased branch, deleted the branch on the repository here, and pushed a new (version of) branch here.

Think this resolves the previous issue as well as issue #6.

craw-daddy commented 6 years ago

And (of course) now the tests are failing. So I've probably done my rebasing incorrectly... Or something else. In case you can't tell, I'm a relative newbie in terms of collaborative development.

ZachGlassman commented 6 years ago

the rebase worked, tests are failing. Remember you can click on the details link and see the test failures on CI. The test that is failing is the pylint code style test, one of your functions does not have a docstring. Fix that and I believe this will be ready to merge.