-
Notifications
You must be signed in to change notification settings - Fork 24
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
Update boolean transformer #228
Conversation
55437c8
to
5f994f9
Compare
caa22d5
to
9cbd6d2
Compare
Codecov Report
@@ Coverage Diff @@
## v0.6.0-dev #228 +/- ##
==============================================
+ Coverage 93.07% 93.45% +0.37%
==============================================
Files 9 9
Lines 650 672 +22
==============================================
+ Hits 605 628 +23
+ Misses 45 44 -1
Continue to review full report at Codecov.
|
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.
LGTM!
* Update boolean transformer * Adapt the BooleanTransformer to the new BaseTransformer * Temporarily support both new and old style transformers * Make columns attributes public * Temporarily support old and new style datasets * Test new-style transformers properly * Temporarily disable W0212 PyLint warning Co-authored-by: Carles Sala <carles@pythiac.com>
* Update boolean transformer * Adapt the BooleanTransformer to the new BaseTransformer * Temporarily support both new and old style transformers * Make columns attributes public * Temporarily support old and new style datasets * Test new-style transformers properly * Temporarily disable W0212 PyLint warning Co-authored-by: Carles Sala <carles@pythiac.com>
No description provided.