Script to generate pyauto documentation.

Script to generate pyauto documentation.
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=98369

[Nirnimesh, 2011-08-26 00:50:17]

Minor comments only

http://codereview.chromium.org/7745039/diff/1/chrome/test/pyautolib/generate_...
File chrome/test/pyautolib/generate_docs.py (right):

http://codereview.chromium.org/7745039/diff/1/chrome/test/pyautolib/generate_...
chrome/test/pyautolib/generate_docs.py:11: 
2 lines at global scope please

http://codereview.chromium.org/7745039/diff/1/chrome/test/pyautolib/generate_...
chrome/test/pyautolib/generate_docs.py:16: help='Directory path to write all of
the documentation')
Add: Defaults to "pyauto_docs" in current directory

http://codereview.chromium.org/7745039/diff/1/chrome/test/pyautolib/generate_...
chrome/test/pyautolib/generate_docs.py:17: parser.add_option('-p', '--pyauto',
dest='pyauto', metavar='FILE',
recommend renaming pyauto to pyautolib

http://codereview.chromium.org/7745039/diff/1/chrome/test/pyautolib/generate_...
chrome/test/pyautolib/generate_docs.py:50: print('Export complete, files are
located in %s' % options.dir)
Remove the parens in all print statements

http://codereview.chromium.org/7745039/diff/1/chrome/test/pyautolib/generate_...
chrome/test/pyautolib/generate_docs.py:51: 
need 2 lines here

[krisr, 2011-08-26 01:02:07]

All done, PTAL.

On Thu, Aug 25, 2011 at 5:50 PM, <nirnimesh@chromium.org> wrote:

> Minor comments only
>
>
> http://codereview.chromium.**org/7745039/diff/1/chrome/**
>
test/pyautolib/generate_docs.**py<http://codereview.chromium.org/7745039/diff/1/chrome/test/pyautolib/generate_docs.py>
> File chrome/test/pyautolib/**generate_docs.py (right):
>
> http://codereview.chromium.**org/7745039/diff/1/chrome/**
>
test/pyautolib/generate_docs.**py#newcode11<http://codereview.chromium.org/7745039/diff/1/chrome/test/pyautolib/generate_docs.py#newcode11>
> chrome/test/pyautolib/**generate_docs.py:11:
> 2 lines at global scope please
>
> http://codereview.chromium.**org/7745039/diff/1/chrome/**
>
test/pyautolib/generate_docs.**py#newcode16<http://codereview.chromium.org/7745039/diff/1/chrome/test/pyautolib/generate_docs.py#newcode16>
> chrome/test/pyautolib/**generate_docs.py:16: help='Directory path to write
> all of the documentation')
> Add: Defaults to "pyauto_docs" in current directory
>
> http://codereview.chromium.**org/7745039/diff/1/chrome/**
>
test/pyautolib/generate_docs.**py#newcode17<http://codereview.chromium.org/7745039/diff/1/chrome/test/pyautolib/generate_docs.py#newcode17>
> chrome/test/pyautolib/**generate_docs.py:17: parser.add_option('-p',
>
> '--pyauto', dest='pyauto', metavar='FILE',
> recommend renaming pyauto to pyautolib
>
> http://codereview.chromium.**org/7745039/diff/1/chrome/**
>
test/pyautolib/generate_docs.**py#newcode50<http://codereview.chromium.org/7745039/diff/1/chrome/test/pyautolib/generate_docs.py#newcode50>
> chrome/test/pyautolib/**generate_docs.py:50: print('Export complete, files
>
> are located in %s' % options.dir)
> Remove the parens in all print statements
>
> http://codereview.chromium.**org/7745039/diff/1/chrome/**
>
test/pyautolib/generate_docs.**py#newcode51<http://codereview.chromium.org/7745039/diff/1/chrome/test/pyautolib/generate_docs.py#newcode51>
> chrome/test/pyautolib/**generate_docs.py:51:
> need 2 lines here
>
>
>
http://codereview.chromium.**org/7745039/<http://codereview.chromium.org/7745...
>

[Nirnimesh, 2011-08-26 01:10:47]

LGTM.

Thanks for doing this.
Time to update the documentation here:
http://dev.chromium.org/developers/testing/pyauto#TOC-Documentation