#9138: Categories for all rings
--------------------------+-------------------------------------------------
Reporter: jbandlow | Owner: nthiery
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.7.2
Component: categories | Keywords: introspection, categories for
rings
Work_issues: | Upstream: N/A
Reviewer: | Author: Simon King
Merged: | Dependencies: sage-4.7, #11268, #11139, #9976,
#9944, #11269, #11316
--------------------------+-------------------------------------------------
Comment(by mderickx):
I see you are the author of both so I guess it depends on wich ticket you
want to get merged first. You should rebase one of the two tickets to
apply cleanly after the other and add the corresponding dependency in the
dependencies field. And maybe also add a note in the description for
possible reviewers that it is wiser to first review the ticket on wich it
depends.
Personally I would want this ticket to be merged before the other. The
main reason is that this ticket really cleans up a lot of rings stuff, and
the other ticket just makes things faster. And in general I prefer to do
things right first and then make them fast ;). The other reason is that I
expect that this ticket will cause more conflicts with other tickets. I
would try to get this one in as fast as possible so there are less patches
merged depending on doing things the old "wrong" way.
Or an even smarter thing to do would be to make both tickets apply cleanly
on their own and passing all doctest in such a way that the can also be
applied at the same time (but maybe not passing all doctests) . Then
create a third patch wich is fixes the conflict between these tickets.
This patch can then be added on wathever ticket gets reviewed/merged last.
In this way none of the two tickets can delay the merging of the other.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9138#comment:97>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
