3 commits

Author	SHA1	Message	Date
Manish V Badarkhe	8671000ff7	docs: add guidelines for abandoning patches ... The code review guidelines have been updated to explain when patches that do not receive a response to the review comments will be abandoned. Change-Id: I60539e16ca41245cf1b352f24557be1b3c67c367 Signed-off-by: Manish V Badarkhe <Manish.Badarkhe@arm.com>	2023-07-06 11:20:28 +01:00
Sandrine Bailleux	ca4febac0c	docs: consolidate code review process documentation ... From the page listing the maintainers and code owners [1], add a link to the code review guidelines page [2], which in turn has a link to the tf.org code review process [3]. Before that patch, both pages [1] and [2] had a link to [3]. Hopefully, this change will guide the reader better so they don't miss out on any information. Additionally, move some of the information from the top of page [1] into page [2] and add extra details about the code review process used in TF-A and how that get translated in Gerrit. [1] https://trustedfirmware-a.readthedocs.io/en/latest/about/maintainers.html [2] https://trustedfirmware-a.readthedocs.io/en/latest/process/code-review-guidelines.html [3] https://developer.trustedfirmware.org/w/collaboration/project-maintenance-process/ Signed-off-by: Sandrine Bailleux <sandrine.bailleux@arm.com> Change-Id: I56562a72443f03fff16077dadc411ef4ee78666d	2023-06-01 13:41:09 +02:00
Sandrine Bailleux	1f19411a14	docs: code review guidelines ... Document the code review process in TF-A. Specifically: * Give an overview of code review and best practices. * Give guidelines for the participants in code review. * Outline responsibilities of each type of participant. * Explain the Gerrit labels used in the review process. Change-Id: I519ca4b2859601a7b897706e310f149a0c92e390 Signed-off-by: Sandrine Bailleux <sandrine.bailleux@arm.com> Signed-off-by: David Horstmann <david.horstmann@arm.com>	2020-10-05 11:54:48 +01:00