Closed chhaylatte closed 7 years ago
Lots of updates. Needs review.
LGTM, any other comments from @rweindl ?
Do I wait for approval from more people or do I just merge it?
It looks fine to me, but I'm not a python guy so I figured it might be good to have @rweindl have the final word, since he created this initially too. It's not blocking anything at the moment, so merging ASAP might not be necessary.
Hey looks also fine for me! Good job. :)
/cc @rweindl @pkluz