-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tables.columnnames(table)
=> Tables.columnnames(cols)
#63
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for fixing all these cases @eliascarv , can we also add a simple test for each transform using a row table? That way we make sure that we don't break things in the future.
Note: I didn't add test for Select and Reject because their fix depends on PR #64. |
Codecov Report
@@ Coverage Diff @@
## master #63 +/- ##
==========================================
- Coverage 92.46% 92.36% -0.10%
==========================================
Files 19 19
Lines 491 498 +7
==========================================
+ Hits 454 460 +6
- Misses 37 38 +1
Continue to review full report at Codecov.
|
closes #61