Hi Varun,
On 7/6/20 7:45 PM, Varun Wadekar wrote:
Hello Sandrine,
IMO, the code review process has now evolved from the previous version. I am not too clear on the final outcome.
I am not sure we've reached a final outcome just yet (see Julius' email). The way we implement it for TF-A is still up for discussion. Once we've reached an agreement, I plan to write it down in the TF-A documentation and upload it as a patch to review in Gerrit so we can double-check we are all on the same page.
Does it make sense to add a "lifecycle of a review" section to the initial proposal?
When you say "initial proposal", are you referring to the generic proposal for tf.org projects? https://developer.trustedfirmware.org/w/collaboration/project-maintenance-pr...
If yes, then there is already a section called "Life Cycle of a Patch -
From submission to integration in the tree" which attempts to cover
this. This outlines the general policy but the details are up for each project to define (which is what we are trying to do right now).
Regards, Sandrine