← Back to team overview

yellow team mailing list archive

Re: Update documentation (issue 6945058)

 

Thanks Francesco.  Good catches.  I ran the files through rst2html a few
times and fixed some other bits I saw too.  I forget how easy it is to
actually try the ReST out. :-P

Gary


https://codereview.appspot.com/6945058/diff/1/HACKING.txt
File HACKING.txt (right):

https://codereview.appspot.com/6945058/diff/1/HACKING.txt#newcode19
HACKING.txt:19: `getting started
<https://juju.ubuntu.com/docs/getting-started.html>`. If you
On 2012/12/14 17:46:51, frankban wrote:
> Missing underscore at the end of the rst link.

Done.

https://codereview.appspot.com/6945058/diff/1/HACKING.txt#newcode108
HACKING.txt:108: If Jitsu generates errors about not being able to
bootstrap...::
Changed to ...\n:: for better rendering.

https://codereview.appspot.com/6945058/diff/1/README.txt
File README.txt (right):

https://codereview.appspot.com/6945058/diff/1/README.txt#newcode33
README.txt:33: and then run the usual bootstrap command.
Added a code block here too.

https://codereview.appspot.com/6945058/diff/1/README.txt#newcode40
README.txt:40: $ juju deploy cs:~juju-gui/precise/juju-gui
On 2012/12/14 17:46:51, frankban wrote:
> Should this be a code block?

Done.

https://codereview.appspot.com/6945058/diff/1/README.txt#newcode49
README.txt:49: $ watch juju status
On 2012/12/14 17:46:51, frankban wrote:
> Same as above.

Done.

https://codereview.appspot.com/6945058/diff/1/README.txt#newcode52
README.txt:52: something that looks like this:
On 2012/12/14 17:46:51, frankban wrote:
> We should add another colon here and indent the following juju status
output
> example, so that rst, considering that a code block, can correctly
compile the
> file.

Done.

https://codereview.appspot.com/6945058/diff/1/README.txt#newcode87
README.txt:87: For now, though, install Jitsu...::
Changed this to the following, because it renders better (the "::" is
omitted)

For now, though, install Jitsu...

::

https://codereview.appspot.com/6945058/

-- 
https://code.launchpad.net/~gary/charms/precise/juju-gui/trunk/+merge/139968
Your team Juju GUI Hackers is requested to review the proposed merge of lp:~gary/charms/precise/juju-gui/trunk into lp:~juju-gui/charms/precise/juju-gui/trunk.


References