FIX: undefined variable

[mgxd]

addresses #1632

[codecov-io]

Current coverage is 70.88% (diff: 0.00%)

Merging #1633 into master will decrease coverage by 0.01%

@@             master      #1633   diff @@
==========================================
  Files          1019       1019          
  Lines         51277      51278     +1   
  Methods           0          0          
  Messages          0          0          
  Branches       7265       7265          
==========================================
- Hits          36351      36346     -5   
- Misses        13840      13851    +11   
+ Partials       1086       1081     -5   

Powered by Codecov. Last update 0bb9561...2ed433d

[mgxd]

I'm stuck on codecov/patch - what am I missing?

[oesteban]

I think you should not worry about the codecov/patch check, we are still figuring out how to integrate codecov correctly.

[mgxd]

@satra @chrisfilo @oesteban ready to merge?

[satra]

testing this would require a real test on real data. this needs to happen across all interfaces.

@mgxd - could you add real testing to the test issue ( #1648 ) that @shoshber created