#9054: create a class for basic function_field arithmetic for Sage
-----------------------------------------------------------------------------------------------+
    Reporter:  was                                                              
               |         Owner:  was                          
        Type:  enhancement                                                      
               |        Status:  needs_work                   
    Priority:  major                                                            
               |     Milestone:  sage-pending                 
   Component:  algebra                                                          
               |    Resolution:                               
    Keywords:                                                                   
               |   Work_issues:                               
    Upstream:  N/A                                                              
               |      Reviewer:  Maarten Derickx, Julian Rueth
      Author:  William Stein, Robert Bradshaw, Maarten Derickx, Moritz 
Minzlaff, Julian Rueth  |        Merged:                               
Dependencies:  #9094, #11751, #9138                                             
               |  
-----------------------------------------------------------------------------------------------+
Changes (by jdemeyer):

  * status:  positive_review => needs_work


Comment:

 The commit messages of the patches could be cleaned up:
  1. [attachment:trac_9054-julian-combined.patch]: the commit message
 ''starts'' with {{{* * *}}} instead of something useful.
  1. [attachment:trac_9054-review.patch] has no proper commit message.
 This improper commit message is also in [attachment:trac_9054-all-
 parts.patch], which should be fixed.
  1. [attachment:trac_9054-all-parts.patch] "{{{contains parts 1-12,
 marteen's additions and final doctest fixes}}}" makes no sense if you
 don't know this ticket, the message should makes sense on its own.  The
 word "function field" does not even appear in the message of this patch!

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9054#comment:80>
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.

Reply via email to