Add a "reviewed by" to each item in the changelog report, listing each reviewer of a merged PR

[choldgraf]

Currently, we add the author name for all PRs that have been merged. However, getting something merged always requires at least two roles: an author and a reviewer.

In the spirit of surfacing more credit for the work people do around pull requests, we could add the option of including reviewer name with each PR item. For example:

Doing something cool #119 (@authorname, reviewed by @reviewer1, @reviewer2)

Implementation

I think we're already grabbing PR reviewer information here:

github-activity/github_activity/graphql.py

Lines 27 to 38 in a9d85f4

	reviews_query = """\
	reviews(last: 100) {
	edges {
	node {
	authorAssociation
	author {
	login
	}
	}
	}
	}
	"""

And somewhere around here I think we'd need to add the reviewer information, maybe behind a feature flag:

github-activity/github_activity/github_activity.py

Line 656 in a9d85f4

for kind, info in prs.items():

[manics]

Isn't this already implemented, though without the reviewed by text?
https://github.com/jupyterhub/jupyterhub/blob/5.3.0/docs/source/reference/changelog.md

[choldgraf]

That is definitely possible! I glanced at the docs but didn't find this, so I just assumed it wasn't there and wanted to write the idea down.

If it is already implemented then I think we can close this by just documenting intended behavior clearly here

[choldgraf]

Threw together a quick PR to try and fix this

• [DOC] Clarify some intended functionality in docs #127