3289: Support for pre-post "hooks"

kevinmm********@gmai***** (Google Code) (Is this you? Claim this profile.) 
What version are you running?
N/A

What's the URL of the page this enhancement relates to, if any?
N/A

Describe the enhancement and the motivation for it.
Would like the ability to run "hooks" in RBTools before posting a new or updated review.  For example, to run a linter or style checker.  Git allows for both global and per-repository (shared) hooks to be defined.  We can add a pre-push or pre-commit hook to do the check, but doing it at review time would also be useful.

The idea would be something like defining a hook in .git/pre-review or .rbtools/pre-review that can cause the post to fail depending on the return value of the hook.  Like Git hooks, this would be both per developer desktop or per repository.

I would imagine there would be other use-cases for this as well.

What operating system are you using? What browser?
Ubuntu Precise, N/A

Please provide any additional information below.
http://git-scm.com/book/en/Customizing-Git-Git-Hooks
#1 kevinmm********@gmai***** (Google Code) (Is this you? Claim this profile.) 
A few use case examples: http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts

Paraphrased:

Ensuring the commit message matches a predetermined style.
Ensuring a linter is run.
Enforcing coding conventions.
Running a dependency checker.
Warning you if you haven't built your client and/or run unit tests within the last X hours.
david
#2 david
  • +Component-RBTools
chipx86
#3 chipx86
  • +Project-RBTools
david
#4 david
  • -reviewboard
    +rbtools
  • -Project:RBTools