Summary

Add Gerrit repository configuration documentation

Review Request #9811 — Created March 19, 2018 and submitted March 19, 2018, 8:35 p.m.

Information

Owner

brennie

Repository

Review Board

Branch

release-3.0.x

Bugs

Depends On

Commit

cf45cd2...

Reviewers

Groups

reviewboard

People

Description

This patch adds detailed installation instructions for Gerrit
repositories.

Testing Done

Built the docs and looked through them.

Issues

Description	From	Last Updated
Your GitLab changes are showing up in this change.	chipx86	March 19, 2018, 7:32 p.m.
"Gerrit"	chipx86	March 19, 2018, 5:18 p.m.
No _ after plugin. No comma before "which". Also might as well just have the link name be Gerrit plugin …	chipx86	March 19, 2018, 5:19 p.m.
Probably no "the" before "Gerrit".	chipx86	March 19, 2018, 5:19 p.m.
Should use :file: for the path.	chipx86	March 19, 2018, 5:19 p.m.
:guilabel: and ->	chipx86	March 19, 2018, 5:20 p.m.
:guilabel: and ->	chipx86	March 19, 2018, 5:20 p.m.
Only one blank line.	chipx86	March 19, 2018, 5:20 p.m.
This is pretty sparse. Can you use the text we use for, say, AWS CodeCommit here?	chipx86	March 19, 2018, 5:21 p.m.
Gerrit.	chipx86	March 19, 2018, 5:20 p.m.

flake8 passed.

JSHint passed.

docs/manual/admin/configuration/repositories/gerrit.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
"Gerrit"
```

docs/manual/admin/configuration/repositories/gerrit.rst (Diff revision 1)

The issue has been resolved. Show all issues

No _ after plugin.

No comma before "which".

Also might as well just have the link name be Gerrit plugin so you don't have to give it an explicit title here.

docs/manual/admin/configuration/repositories/gerrit.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
Probably no "the" before "Gerrit".
```
docs/manual/admin/configuration/repositories/gerrit.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
Should use :file: for the path.
```
docs/manual/admin/configuration/repositories/gerrit.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
:guilabel: and ->
```
docs/manual/admin/configuration/repositories/gerrit.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
:guilabel: and ->
```
docs/manual/admin/configuration/repositories/gerrit.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
Only one blank line.
```
docs/manual/admin/configuration/repositories/gerrit.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
This is pretty sparse. Can you use the text we use for, say, AWS CodeCommit here?
```
docs/manual/admin/configuration/repositories/gerrit.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
Gerrit.
```

Commit:

-	d4d9ad6aff844d81930ef17cf0e04cd183914ed1
+	b04b6b8aa887c24e0944aa704ed170415afff71b

Diff:

Revision 2 (+122 -7)

Show changes

	docs/manual/admin/index.rst
	docs/manual/admin/configuration/repositories/gerrit.rst
	docs/manual/admin/configuration/repositories/gitlab.rst
	docs/manual/admin/configuration/repositories/index.rst
	reviewboard/hostingsvcs/gerrit.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

The issue has been resolved. Show all issues

Your GitLab changes are showing up in this change.

Commit:

-	b04b6b8aa887c24e0944aa704ed170415afff71b
+	cf45cd2a08548a04f314aae000af9b25b241f0e0

Diff:

Revision 3 (+113 -1)

Show changes

	docs/manual/admin/index.rst
	docs/manual/admin/configuration/repositories/gerrit.rst
	docs/manual/admin/configuration/repositories/index.rst
	reviewboard/hostingsvcs/gerrit.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (821f7d2)