Skip to content

Conversation

@glemaitre
Copy link
Member

Reference Issue

What does this implement/fix? Explain your changes.

Allows pandas series in/out for y

Any other comments?

@pep8speaks
Copy link

pep8speaks commented Nov 17, 2019

Hello @glemaitre! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2019-11-17 23:06:24 UTC

@glemaitre glemaitre merged commit f356284 into scikit-learn-contrib:master Nov 17, 2019
@lgtm-com
Copy link

lgtm-com bot commented Nov 17, 2019

This pull request introduces 1 alert when merging 1b4aa9c into 158258e - view on LGTM.com

new alerts:

  • 1 for Mismatch in multiple assignment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants