Summary

Add a resource for retrieving a list of commits

Review Request #8922 — Created May 5, 2017 and submitted May 23, 2017, 3:15 p.m.

Information

Owner

brennie

Repository

gerrit-reviewboard-plugin

Branch

master

Bugs

Depends On

~~8920~~

Blocks

8923

Reviewers

Groups

gerrit-reviewboard-plugin

People

Description

The CommitListResource, available at
a/projects/<projectName>/all-commits/, returns a list of commit
information entries about the first N (default 30) commits on a
branch. It also supports starting at a specific commit for
pagination support.

Testing Done

Hit the API with ?limit and ?start queries and saw the correct
results.

Issues

Description	From	Last Updated
Inconsistent spacing around =s. Shouldn't have any spaces, I think.	david	May 11, 2017, 8:58 a.m.
Inconsistent spacing around =s. Shouldn't have any spaces, I think. Also, CNT?	david	May 11, 2017, 8:58 a.m.
Should we be doing some kind of logging rather than just saying to print to console?	david	May 11, 2017, 8:58 a.m.
What happens if the commit has no parents?	david	May 22, 2017, 10:45 p.m.
Wrap this?	david	May 11, 2017, 8:58 a.m.

JSHint passed.

PEP8 Style Checker passed.

Pyflakes passed.

Diff:

Revision 2 (+158)

Show changes

	src/main/java/org/reviewboard/rbgerrit/CommitListResource.java
	src/main/java/org/reviewboard/rbgerrit/Module.java

Checks run (3 succeeded)

JSHint passed.

PEP8 Style Checker passed.

Pyflakes passed.

Change Summary:


Move the resource under the project
add full message
add author time

Description:

		The `CommitListResource`, available at
~		`a/projects/<projectName>/branches/<branchName>/commits/`, returns a
~		list of commit information entries about the first N (default 30)
~		commits on a branch. It also supports starting at a specific commit
~		for pagination support.
	~	`a/projects/<projectName>/all-commits/`, returns a list of commit
	~	information entries about the first N (default 30) commits on a
	~	branch. It also supports starting at a specific commit for
	~	pagination support.