-
-
Notifications
You must be signed in to change notification settings - Fork 567
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
More doctests for sage.rings #13685
Comments
This comment has been minimized.
This comment has been minimized.
Changed dependencies from #12496 to none |
comment:2
Rebased to |
Dependencies: #13618 |
Reviewer: Volker Braun |
comment:4
Looks good to me. |
comment:6
Thank you for reviewing this, Travis |
comment:7
Needs to be rebased for sage-5.8.beta1, I get
(we don't like fuzz > 1) |
comment:8
I'll build |
comment:9
And I'll also rebase over #14040. |
comment:10
In fact, better to wait for sage-5.8.beta2 as there are more patches to rings with positive review. |
comment:11
Replying to @jdemeyer:
Will do. |
Rebased to 5.9.beta1 |
comment:12
Attachment: trac_13685-misc_rings_doc-ts.patch.gz Rebased to |
Merged: sage-5.9.beta2 |
More doctests for the rest of
sage.rings
as a followup to #13618. This adds documentation for minor functions in the following files:Most of these files were a few functions short of 100% doctest coverage. This patch also goes through and cleans up some of the documentation in these files as well.
Depends on #13618
Depends on #14040
Component: doctest coverage
Author: Travis Scrimshaw
Reviewer: Volker Braun
Merged: sage-5.9.beta2
Issue created by migration from https://trac.sagemath.org/ticket/13685
The text was updated successfully, but these errors were encountered: