project-hatohol/hatohol — pull request #65 by okkez (2013-09-30 15:47:33+00:00)
Title: Use argparse

Use argparse standard library.

2 Incompatibility changes as followings

Add `--to-address` required option

Before:
````
$ hatohol-actor-mail to@example.com --hatohol-action-v1 1 2 3 456.789 1 2 3 4 5
````
After:
````
$ hatohol-actor-mail --to-address to@example.com --hatohol-action-v1 1 2 3 456.789 1 2 3 4 5
````
Add `--smtp-port` optional option

Before:
````
$ hatohol-actor-mail --smtp-server mail.example.com 587 --tls user password \
to@example.com --hatohol-action-v1 1 2 3 456.789 1 2 3 4 5
````
After:
````
$ hatohol-actor-mail --smtp-server mail.example.com --smtp-port 587 --tls user password \
--to-address to@example.com --hatohol-action-v1 1 2 3 456.789 1 2 3 4 5
````

I think that there are some TODOs in this script.

* Specify desctiprion to ArgumentParser's parameter
* I couldn't find proper description for this script
* I think `--tls` option should be flag option
* And add 2 options to specify ACCOUNT and PASSWORD such as `--smtp-user` and `--smpt-password`Incompatibility changes as followings