Update hacking.rst to mention reviews.

Update hacking.rst to mention reviews.

Both Launchpad-only and lbox reviews are detailed.

https://code.launchpad.net/~bac/charmworld/update-hacking/+merge/193298

(do not edit description out of merge proposal)

[bac, 2013-10-30 18:03:04]

Please take a look.

[benji, 2013-10-30 18:16:58]

LGTM with a nit.

https://codereview.appspot.com/19890043/diff/1/docs/hacking.rst
File docs/hacking.rst (right):

https://codereview.appspot.com/19890043/diff/1/docs/hacking.rst#newcode42
docs/hacking.rst:42: ========================
+1

https://codereview.appspot.com/19890043/diff/1/docs/hacking.rst#newcode107
docs/hacking.rst:107: room.  Find us there if you'd like to contribute changes
back to the project.
"room"?  You know better than that. ;P

https://codereview.appspot.com/19890043/diff/1/docs/hacking.rst#newcode110
docs/hacking.rst:110: done solely on Launchpad as `merge proposals` or by using
the `lbox` tool to
In reST, backticks apply the default role to text, with the default being
:title-reference:.  I don't think that is what you meant here.  I suggest using
double-quotes instead.

[bac, 2013-10-30 18:46:27]

Please take a look.

[bac, 2013-10-30 18:57:10]

thanks for the review

[benji, 2013-10-30 19:06:51]

This looks even more LGTM that it did before.