Review Board

beta

Add configuration variables to the top of post-review for site-wide defaults

Submitted
Updated 7 months, 3 weeks ago

Marc Hedlund Reviewers
reviewboard
None Review Board SVN
This adds a set of configuration variables to the top of post-review, one for each of the command-line switches where it would make sense, so that sites can easily set site-wide defaults.  

I also made the command-line option definitions more consistent, and added '-n' as a short flag for the '--output-diff' option (since '-n' usually means "tell me what you're going to do without actually doing it," which seems appropriate here and is how I use that option).

Finally, I updated the usage string.
I've done basic sanity-checking on each of the configuration variables.
Ship it!
Posted 7 months, 3 weeks ago (April 1st, 2008, 2:10 a.m.)
This looks great. I'll commit this and your other change and bump the version to 0.7.

Thanks!
Ship it!
Posted 7 months, 3 weeks ago (April 1st, 2008, 2:30 a.m.)
Committed in r1257. Thanks again! Excellent work on these patches.