Summary

Add --ignore-submodules to postreview.py

Review Request #3052 — Created April 9, 2012 and submitted April 9, 2012, 6:26 p.m.

Information

Owner

ayust

Repository

RBTools

Branch

Bugs

1364

Depends On

Reviewers

Groups

rbtools

People

david

Description

RB doesn't handle submodules; diffs that include submodule changes break diff uploading. Thus, postreview.py shouldn't include them.

https://github.com/reviewboard/rbtools/pull/9

Testing Done

We use this patch in production at Yelp.

Expand All
Collapse All

Ship it!

```
Looks good. Thanks for the change!
```

Status:: Completed
Change Summary:: Pushed to master (a5e95a2)