#9771: update instructions for reviewers, especially for new developers
-------------------------------+--------------------------------------------
Reporter: niles | Owner: mvngu
Type: enhancement | Status: positive_review
Priority: major | Milestone: sage-4.6
Component: documentation | Keywords: doctests, reviewing
Author: Niles Johnson | Upstream: N/A
Reviewer: Minh Van Nguyen | Merged:
Work_issues: |
-------------------------------+--------------------------------------------
Changes (by newvalueoldvalue):
* status: needs_review => positive_review
* reviewer: => Minh Van Nguyen
* milestone: => sage-4.6
* author: => Niles Johnson
Comment:
Applies fine to Sage 4.5.3.rc0. No doctests were introduced, deleted, or
modified so there is no need for me to run any doctests. I like the
changes that cater to people new to Sage development. One could search for
tickets needing review that have the priority "minor". We also have the
priority "trivial". You could add that in with your current patch if you
wish. Otherwise, you could open a new ticket to mention this point. In any
case, I give the current patch a positive review.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9771#comment:2>
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.
