Code review comment for lp:~frankban/juju-gui/release-process-fixes

Revision history for this message
Francesco Banconi (frankban) wrote :

Thanks for the review Nicola.

https://codereview.appspot.com/7039049/diff/1/docs/process.rst
File docs/process.rst (right):

https://codereview.appspot.com/7039049/diff/1/docs/process.rst#newcode109
docs/process.rst:109: - If you are using a pre-existing branch, make
sure it is up-to-date::
On 2013/01/03 12:06:30, teknico wrote:
> No need for double colons in the two lines above.

Done.

https://codereview.appspot.com/7039049/diff/1/docs/process.rst#newcode128
docs/process.rst:128: - In the ``build-prod`` directory, inside the
uncompressed one, start a server:
On 2013/01/03 12:06:30, teknico wrote:
> s/inside/under/

Done.

https://codereview.appspot.com/7039049/diff/1/docs/process.rst#newcode183
docs/process.rst:183: - If you are using a pre-existing branch, make
sure it is up-to-date::
On 2013/01/03 12:06:30, teknico wrote:
> No need for double colons in the two lines above.

Done.

https://codereview.appspot.com/7039049/

« Back to merge proposal