History log of /rk3399_ARM-atf/docs/process/code-review-guidelines.rst (Results 1 – 8 of 8)
Revision Date Author Comments
# 4d64be30 27-Dec-2023 Sandrine Bailleux (on vacation) <sandrine.bailleux@arm.com>

Merge "docs: update links to tf.org-wide process documents" into integration


# 979c5482 21-Dec-2023 Sandrine Bailleux <sandrine.bailleux@arm.com>

docs: update links to tf.org-wide process documents

tf.org-wide documents have been migrated away from
developer.trustedfirmware.org, because the latter will be
decomissioned at some point in the fu

docs: update links to tf.org-wide process documents

tf.org-wide documents have been migrated away from
developer.trustedfirmware.org, because the latter will be
decomissioned at some point in the future. These documents are now
hosted in a new 'tf_docs' repository hosted on Github [1] and can be
easily browsed through a new ReadTheDocs website at [2].

Update all relevant links in TF-A documentation to refer to [2].

[1] https://github.com/TrustedFirmware/tf_docs
[2] https://trusted-firmware-docs.readthedocs.io/

Signed-off-by: Sandrine Bailleux <sandrine.bailleux@arm.com>
Change-Id: Ib9d39c36250a05754fe5e46cb6f3044ecb776534

show more ...


# e318411f 07-Jul-2023 Olivier Deprez <olivier.deprez@arm.com>

Merge "docs: add guidelines for abandoning patches" into integration


# 8671000f 04-Jul-2023 Manish V Badarkhe <Manish.Badarkhe@arm.com>

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:

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>

show more ...


# ff31094a 01-Jun-2023 Sandrine Bailleux <sandrine.bailleux@arm.com>

Merge changes from topic "sb/maintainters-process" into integration

* changes:
docs: clarify maintainers election process
docs: consolidate code review process documentation


# ca4febac 25-May-2023 Sandrine Bailleux <sandrine.bailleux@arm.com>

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.o

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

show more ...


# bd260fcb 20-Oct-2020 Manish Pandey <manish.pandey2@arm.com>

Merge "docs: code review guidelines" into integration


# 1f19411a 17-Aug-2020 Sandrine Bailleux <sandrine.bailleux@arm.com>

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.

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>

show more ...