Files
training-guides/doc/upstream-training/source/slides/workflow-reviewing.rst
Kendall Nelson 81642be7a9 [upstream] Remove Tags
This patch removes the tags table from the site and tags from the
slides. Since the content has moved out of the slides and into the
contributor guide, the tags are not really useful anymore. Moreover,
they weren't being maintained as new slide decks were added anyway.

Change-Id: I7f10d47364e3792177e34f915280c2ae025d65d9
2018-10-05 16:03:57 -07:00

29 lines
683 B
ReStructuredText

=========
Reviewing
=========
.. image:: ./_assets/os_background.png
:class: fill
:width: 100%
Learning Exercise
=================
* Open the `Using Gerrit
<https://docs.openstack.org/contributors/code-and-documentation/using-gerrit.html>`_
section of the Contributor Guide
* Read the material
* Ask the mentors questions
* Get ready to go through the exercises
Exercise
========
Review three patches in the sandbox environment:
https://docs.openstack.org/contributors/code-and-documentation/sandbox-house-rules.html
Try to find things to make comments on even if they
are just asking a question and not pointing out an
issue, don't just +1 three different patches.