#8607: add deprecation option to the plot option rename decorator
-------------------------------+--------------------------------------------
   Reporter:  jason            |       Owner:  was            
       Type:  enhancement      |      Status:  positive_review
   Priority:  major            |   Milestone:  sage-4.4       
  Component:  graphics         |    Keywords:                 
     Author:  Jason Grout      |    Upstream:  N/A            
   Reviewer:  Sébastien Labbé  |      Merged:                 
Work_issues:                   |  
-------------------------------+--------------------------------------------
Changes (by slabbe):

  * status:  needs_review => positive_review


Comment:

 I removed the deprecation in the word code from the review patch. I
 realize more and more that mixing independent things in the same ticket is
 not a good idea : the review process gets unnecessarily longer. Hence, I
 kept in the review patch only the documentation changes to the file
 {{{sage/plot/misc.py}}} which was given a positive review by Jason.
 Therefore, this ticket now deserves a positive review.

 Sorry again for the first version of the review patch. I hope it will make
 it into sage-4.4. Thanks again for the idea about this ticket : I like it
 and will definitively use it ...in later patches!

 One last idea (for another ticket) since I am thinking about it. Maybe
 this {{{rename_keyword}}} decorator could move up in {{{sage/misc}}}
 because I think it will be used by more than the plot code.

 Sébastien

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