Skip to content
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

dependencies: sklearn -> scikit-learn #65

Merged
merged 1 commit into from
Jan 11, 2023
Merged

Conversation

vpipkt
Copy link
Contributor

@vpipkt vpipkt commented Jan 2, 2023

declaring sklearn as a dependency is going to be deprecated https://github.com/scikit-learn/sklearn-pypi-package

declaring `sklearn` as a dependency is going to be deprecated
https://github.com/scikit-learn/sklearn-pypi-package
@vpipkt
Copy link
Contributor Author

vpipkt commented Jan 9, 2023

@gkhayes @ChristopherBilg would you kindly take a look? 🙇

@ChristopherBilg
Copy link
Contributor

Yup, good preventative maintenance.

@vpipkt
Copy link
Contributor Author

vpipkt commented Jan 9, 2023

@ChristopherBilg thanks for the attention to it!

@ChristopherBilg
Copy link
Contributor

@vpipkt Anytime!

@vpipkt
Copy link
Contributor Author

vpipkt commented Jan 11, 2023

@ChristopherBilg i don't have premission to merge to this repo. Any action required on my part to get this merged?

@ChristopherBilg
Copy link
Contributor

Nor do I, just need to wait on @gkhayes for that.

@gkhayes gkhayes merged commit d9c2b3e into gkhayes:master Jan 11, 2023
@vpipkt
Copy link
Contributor Author

vpipkt commented Jan 12, 2023

i guess the next question is when we can expect a release with this change 🥺

@vpipkt
Copy link
Contributor Author

vpipkt commented Jan 19, 2023

@gkhayes @ChristopherBilg When can we expect a release with this change?

@ChristopherBilg
Copy link
Contributor

That would be on @gkhayes to bump minor versions.

@vpipkt
Copy link
Contributor Author

vpipkt commented Feb 3, 2023

@gkhayes can we expect a release with this update?

@gkhayes
Copy link
Owner

gkhayes commented Feb 4, 2023

@vpipkt @ChristopherBilg - Sorry about the delay, but I have been busy with my day job. Will try to get something out in the next few weeks, but I can't promise anything.

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.

3 participants