-
-
Notifications
You must be signed in to change notification settings - Fork 401
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
build/pkgs/ipython/dependencies: drop jinja2 #36209
Conversation
Can we also remove it from the run-time dependencies of sagelib? IIRC, it is only used at build time via sage_setup |
Also needs to be removed from the install-requires in src/setup.cfg.m4 |
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, thanks.
merge conflict |
IPython itself no longer depends on jinja2 after nbconvert was factored out of it.
The Sage library needs jinja2 only at build time and only via sage_setup, which is already one of its dependencies.
The Sage library does not itself require jinja2; only sage_setup does.
e151df9
to
6f9fd72
Compare
Documentation preview for this PR (built with commit 6f9fd72; changes) is ready! 🎉 |
IPython itself no longer depends on jinja2 after nbconvert was factored out of it. URL: sagemath#36209 Reported by: Michael Orlitzky Reviewer(s): Matthias Köppe
IPython itself no longer depends on jinja2 after nbconvert was factored out of it.