-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
chore: remove ipython from development dependencies #28703
Conversation
I thought development dependencies were there to help aid development. Maybe it's worth explicitly mentioning in the documentation what are viable development dependencies. |
Yes for this particular one the "likely hood to be used" vs "dependency burden" ratio isn't great. Currently the criteria is more or less "commonly used by humans and/or CI for development workflows". Now that we have a workflow to upgrade things (supersetbot as a pip-compile-multi-aware dependabot replacement), it seems reasonable / manageable if we wanted to bring back more segmentation/complexity under |
One more thought it with segmentation in |
Summary
removing ipython as it's simply a nice-to-have, but not required for Superset development