Skip to content

BLD remove jupyter from requirements#120

Merged
BorisMuzellec merged 3 commits intomainfrom
remove_jupyter_dep
Apr 25, 2023
Merged

BLD remove jupyter from requirements#120
BorisMuzellec merged 3 commits intomainfrom
remove_jupyter_dep

Conversation

@BorisMuzellec
Copy link
Copy Markdown
Collaborator

@BorisMuzellec BorisMuzellec commented Apr 24, 2023

Reference Issue or PRs

Fixes #113

What does your PR implement? Be specific.

Removes jupyter from the setup.py requirements and replaces it with ipython (used in DeseqStats.summary()).

@BorisMuzellec BorisMuzellec requested a review from a user April 24, 2023 08:09
@BorisMuzellec BorisMuzellec changed the title build: remove jupyter from requirements BLD remove jupyter from requirements Apr 24, 2023
Copy link
Copy Markdown
Collaborator

@maikia maikia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@BorisMuzellec BorisMuzellec merged commit 1801cbe into main Apr 25, 2023
@BorisMuzellec BorisMuzellec deleted the remove_jupyter_dep branch April 25, 2023 13:39
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.

Move jupyter to extras_require

2 participants