Summary

Add bazaar support to post-review

Review Request #3424 — Created Oct. 17, 2012 and submitted Oct. 30, 2012, 7:40 p.m.

Owner

Repository

RBTools

Branch

master

Bugs

773

Depends On

Groups

People

Description

I updated the change from http://reviews.reviewboard.org/r/1739/ which added support for Bazaar to post-review, but is now out of date.

Testing Done

Ran the unit tests included, and used it locally to submit some changes.

Issues

Description	From	Last Updated
Please move this comment to be above BRANCH_REGEX, and use #s instead of a triple-quoted string.	david	Oct. 22, 2012, 1:26 a.m.
Remove this line.	david	Oct. 22, 2012, 1:27 a.m.
Remove this line.	david	Oct. 22, 2012, 1:27 a.m.
Remove this line. Same for other docstrings below.	david	Oct. 22, 2012, 1:27 a.m.
Remove this line.	david	Oct. 22, 2012, 1:27 a.m.

+	773

This looks pretty great. I have a handful of style comments.

rbtools/clients/bazaar.py (Diff revision 2)

The issue has been resolved. Show all issues

Please move this comment to be above BRANCH_REGEX, and use #s instead of a triple-quoted string.

rbtools/clients/bazaar.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Remove this line.
```
rbtools/clients/bazaar.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Remove this line.
```
rbtools/clients/bazaar.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Remove this line. Same for other docstrings below.
```
rbtools/clients/tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Remove this line.
```

Cleaned up comments and docstrings

Ship it!

Ship it!

Pushed to master (85f43d5). Thanks!