-
-
Notifications
You must be signed in to change notification settings - Fork 491
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
sage.{calculus,functions,numerical,symbolic}
: Docstring/doctest cosmetics, # needs
#37715
sage.{calculus,functions,numerical,symbolic}
: Docstring/doctest cosmetics, # needs
#37715
Conversation
Documentation preview for this PR (built with commit e56c10b; changes) is ready! 🎉 |
I'm sorry, but I can't review this, because I don't understand modularization. |
e9ff69a
to
ef23716
Compare
@mkoeppe For this business of maintaining the "# needs" or "# optional" tags, I think that it should be obvious to a PR author how to modify or add or delete those tags. I guess that "Test modularized distributions" of "B&T" would (could) annotate "Files changed" with appropriate messages on how to modify them. If I am right, is it working now or are you working for it to work? |
Otherwise, lgtm. |
This can only be done after the distributions in #35095 have been merged into |
I see. Thanks. |
Thank you! |
fail, see CI |
ef23716
to
e56c10b
Compare
Cherry-picked from #35095.
📝 Checklist
⌛ Dependencies