On 2 mars 2014, at 09:59, Josh Smeaton <[email protected]> wrote:
> Is there anyway of indicating that a patch needs a review? Or is simply > setting "ready for checkin" a good enough signal? "Ready for checkin" means that you have reviewed the patch and you think it's complete and correct. If the patch applies cleanly but you haven't looked at it, the correct status is "Accepted" + "has patch". That combination implies that the patch needs a review. -- Aymeric. -- You received this message because you are subscribed to the Google Groups "Django developers" 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/django-developers. To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/01E7E11B-2EFA-4F0E-BCBE-0541B0E5E0C0%40polytechnique.org. For more options, visit https://groups.google.com/groups/opt_out.
