Subject: Commit / Code Review Policy


Some thoughts from reading the doc and this thread
   1. This doesn't sound like a huge change, mostly a change in tone since
   nearly everything in it is a judgement call.
   2. I agree that the word policy seems iffy. It has a "consequences for
   violation" tone to it, and without some sort of "what happens if you break
   it" it's really just a recommendation anyway. Maybe call it "Commit Process
   Recommendations"
   3. It would be nice if JIRA had a (*optional!*) PATCH_REVIEW status
   (down stream from PATCH_AVAILABLE and it's infrastructure, but not
   requiring PATCH_AVAILABLE as a prior step) that could be hooked up to send
   a mail with an easily filterable subject line to individuals that have
   (somehow) nominated themselves as interested in the associated Jira
   Component
   4. If we add the status, we should formalize the expected time period
   that reviewers will have to respond, even if it's just with "I want to
   review this but I can't get to it till Thursday..." (I'm thinking 1 week).
   This timeline would only apply to issues that use this status, timelines
   for small stuff, or major changes etc as per the recommendations... This is
   just an idea to help folks who want a review to connect with and get the
   attention of folks who care particularly deeply about a feature area.
On Mon, Dec 2, 2019 at 6:00 AM Ishan Chattopadhyaya <
[EMAIL PROTECTED]> wrote:

--
http://www.needhamsoftware.com (work)
http://www.the111shift.com (play)