Add unit tests for public API

cashapp / contour

Layouts with lambdas 😎

Apache License 2.0

1.53k stars 50 forks source link

Add unit tests for public API #131

Closed dongheeleeme closed 3 years ago

dongheeleeme commented 3 years ago

I've added unit tests for public API(XFloat, XInt, YFloat, YInt) based on this issue. Do we have some conventions to write unit tests? If so, I'll make changes according to the rules. Thanks 😄

oldergod commented 3 years ago

fixes #128

dongheeleeme commented 3 years ago

Amazing job, thank you.

Could you sign our CLA so I can merge your PR?

Awesome, thank you so much 😄 I just signed CLA form.

oldergod commented 3 years ago

Thanks!