#18228: Developer guide: Do not change tickets with status "positive_review"
-------------------------------------+-------------------------------------
       Reporter:  cheuberg           |        Owner:
           Type:  defect             |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.7
      Component:  documentation      |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Clemens Heuberger  |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:  u/cheuberg/doc     |       Commit:
  /do-not-change-positive-review-    |  e18c88688732d8d26ff3fc4a0723d14e8a37cf16
  tickets                            |     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------
Changes (by cheuberg):

 * status:  needs_info => needs_review


Comment:

 Hello Nathann,

 I had troubles with trac once more, I did not accept my answers, so I
 shortened them until I could press the submit button. So I try not to use
 the reply function, perhaps then trac will accept longer answers.

 You are right, I am not sure whether I understand your first suggestion.

 My understanding from the sage-devel thread is that the release manager
 merges a certain number of positive_review tickets into his private
 develop branch, runs some tests, and if those pass, he then closes the
 tickets without looking at them again. This may take several hours.

 If he'd look at the ticket again and would see a mismatch, he'd have to
 repeat the whole procedure, and this might lead to an infinite loop. And
 in his words, "Having the release manager chase after branches as people
 keep adding stuff after review is not a sustainable workflow".

 Concerning your second point: *I* do not insist on changing the rules. I
 have opened a sage-devel thread, linked above. Several suggestions, some
 of them similar to yours, have been made by several people, but the
 release manager did not show any indication of changing his point of view
 that a ticket with positive review must not be changed.

 I think that the issue has to be resolved as the 7 tickets show.
 Therefore, as the easiest solution, I propose this ticket.

--
Ticket URL: <http://trac.sagemath.org/ticket/18228#comment:13>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to