Update documentation for BattOr device mapping.

Update documentation for BattOr device mapping.

Added additional documentation explaining how to use Battor-to-device
mapping files.

Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/501ed91927682f0fd3160f0371d968424b5cbb62

[alexandermont, 2016-04-05 22:10:07]

alexandermont@chromium.org changed reviewers:
+ jbudorick@chromium.org, rnephew@google.com

[alexandermont, 2016-04-11 23:35:55]

alexandermont@chromium.org changed reviewers:
+ nednguyen@google.com

[alexandermont, 2016-04-11 23:36:10]

+nednguyen

[nednguyen, 2016-04-12 17:37:17]

Description was changed from

==========
Update documentation for BattOr device mapping.

Added additional documentation explaining how to use Battor-to-device
mapping files.
==========

to

==========
Update documentation for BattOr device mapping.

Added additional documentation explaining how to use Battor-to-device
mapping files.
==========

[nednguyen, 2016-04-12 17:37:17]

nednguyen@google.com changed reviewers:
+ charliea@chromium.org

[charliea (OOO until 10-5), 2016-04-14 15:49:35]

lgtm

[nednguyen, 2016-04-14 16:19:17]

Your documentation looks great to me. However, I wonder whether we should put
this as a help string for the script to use.

You can do s.t like this:
<top of the file>
""""
This script provides tools to map BattOrs to USB devices...
"""

then in argparse:
parser = argparse.ArgumentParser(description=__doc__)

https://codereview.chromium.org/1863773002/diff/40001/devil/devil/utils/batto...
File devil/devil/utils/battor_device_mapping.py (right):

https://codereview.chromium.org/1863773002/diff/40001/devil/devil/utils/batto...
devil/devil/utils/battor_device_mapping.py:15: # Phones are identified by the
following string:
nits: Android phone

[rnephew (Reviews Here), 2016-04-14 16:20:50]

rnephew@chromium.org changed reviewers:
+ rnephew@chromium.org

[rnephew (Reviews Here), 2016-04-14 16:20:50]

https://codereview.chromium.org/1863773002/diff/40001/devil/devil/utils/batto...
File devil/devil/utils/battor_device_mapping.py (right):

https://codereview.chromium.org/1863773002/diff/40001/devil/devil/utils/batto...
devil/devil/utils/battor_device_mapping.py:15: # Phones are identified by the
following string:
On 2016/04/14 16:19:17, nednguyen wrote:
> nits: Android phone

Device*

[nednguyen, 2016-04-14 16:22:30]

Err, looks like this file is not a runnable script. Though it's still good to
use pydoc instead of #.

non owern lgtm though

[alexandermont, 2016-04-14 17:30:21]

The CQ bit was checked by alexandermont@chromium.org

[alexandermont, 2016-04-14 17:30:21]

The patchset sent to the CQ was uploaded after l-g-t-m from
nednguyen@google.com, charliea@chromium.org
Link to the patchset: https://codereview.chromium.org/1863773002/#ps60001
(title: " ")

[commit-bot: I haz the power, 2016-04-14 17:30:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1863773002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1863773002/60001

[commit-bot: I haz the power, 2016-04-14 17:55:13]

Description was changed from

==========
Update documentation for BattOr device mapping.

Added additional documentation explaining how to use Battor-to-device
mapping files.
==========

to

==========
Update documentation for BattOr device mapping.

Added additional documentation explaining how to use Battor-to-device
mapping files.

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2016-04-14 17:55:14]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

[nednguyen, 2016-04-14 19:14:09]

On 2016/04/14 17:55:14, commit-bot: I haz the power wrote:
> Committed patchset #4 (id:60001) as
>
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

Sorry, I am in the owner of catapult/ but my stamp was not supposed to replace
the stamp of devil's OWNERs to land this change. This is my bad, can devil's 
owners still take a look?