From 9d3a33f060d9a526b097fbddadfb5265bf3cf3e3 Mon Sep 17 00:00:00 2001 From: Esko Date: Thu, 6 Apr 2017 00:50:08 +0200 Subject: [PATCH 1/3] Use two hyphens for long flags --- pacli/__main__.py | 30 +++++++++++++++--------------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/pacli/__main__.py b/pacli/__main__.py index 7e4a30f..04a870d 100644 --- a/pacli/__main__.py +++ b/pacli/__main__.py @@ -620,28 +620,28 @@ def cli(): dest="command", description='valid subcommands') - parser.add_argument("-newaddress", action="store_true", + parser.add_argument("--newaddress", action="store_true", help="generate a new address and import to wallet.") - parser.add_argument("-status", action="store_true", help="show pacli status.") - parser.add_argument("-addressbalance", action="store", nargs=2, help="check card balance of the address") + parser.add_argument("--status", action="store_true", help="show pacli status.") + parser.add_argument("--addressbalance", action="store", nargs=2, help="check card balance of the address") deck = subparsers.add_parser('deck', help='Deck manipulation.') - deck.add_argument("-list", choices=['all', 'subscribed'], + deck.add_argument("--list", choices=['all', 'subscribed'], help="list decks") - deck.add_argument("-info", action="store", help="show details of ") - deck.add_argument("-subscribe", action="store", help="subscribe to ") - deck.add_argument("-search", action="store", help='''search for decks by name, id, + deck.add_argument("--info", action="store", help="show details of ") + deck.add_argument("--subscribe", action="store", help="subscribe to ") + deck.add_argument("--search", action="store", help='''search for decks by name, id, issue mode, issuer or number of decimals.''') - deck.add_argument("-new", action="store", help="spawn new deck") - deck.add_argument("-checksum", action="store", help="verify deck card balance checksum.") - deck.add_argument("-balances", action="store", help="show balances of this deck.") + deck.add_argument("--new", action="store", help="spawn new deck") + deck.add_argument("--checksum", action="store", help="verify deck card balance checksum.") + deck.add_argument("--balances", action="store", help="show balances of this deck.") card = subparsers.add_parser('card', help='Cards manipulation.') - card.add_argument("-list", action="store", help="list all card transactions of this deck.") - card.add_argument("-export", action="store", nargs=2, help="export all cards of this deck to csv file.") - card.add_argument("-issue", action="store", help="issue cards for this deck.") - card.add_argument("-transfer", action="store", help="send cards to receivers.") - card.add_argument("-burn", action="store", help="burn cards.") + card.add_argument("--list", action="store", help="list all card transactions of this deck.") + card.add_argument("--export", action="store", nargs=2, help="export all cards of this deck to csv file.") + card.add_argument("--issue", action="store", help="issue cards for this deck.") + card.add_argument("--transfer", action="store", help="send cards to receivers.") + card.add_argument("--burn", action="store", help="burn cards.") return parser.parse_args() From 491a58d420cb494d576ded1d0e9d651d35da6698 Mon Sep 17 00:00:00 2001 From: Esko Date: Thu, 6 Apr 2017 00:57:40 +0200 Subject: [PATCH 2/3] Remove full stops in argument descriptions --- pacli/__main__.py | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/pacli/__main__.py b/pacli/__main__.py index 04a870d..c943080 100644 --- a/pacli/__main__.py +++ b/pacli/__main__.py @@ -621,8 +621,8 @@ def cli(): description='valid subcommands') parser.add_argument("--newaddress", action="store_true", - help="generate a new address and import to wallet.") - parser.add_argument("--status", action="store_true", help="show pacli status.") + help="generate a new address and import to wallet") + parser.add_argument("--status", action="store_true", help="show pacli status") parser.add_argument("--addressbalance", action="store", nargs=2, help="check card balance of the address") deck = subparsers.add_parser('deck', help='Deck manipulation.') @@ -631,17 +631,17 @@ def cli(): deck.add_argument("--info", action="store", help="show details of ") deck.add_argument("--subscribe", action="store", help="subscribe to ") deck.add_argument("--search", action="store", help='''search for decks by name, id, - issue mode, issuer or number of decimals.''') + issue mode, issuer or number of decimals''') deck.add_argument("--new", action="store", help="spawn new deck") - deck.add_argument("--checksum", action="store", help="verify deck card balance checksum.") - deck.add_argument("--balances", action="store", help="show balances of this deck.") + deck.add_argument("--checksum", action="store", help="verify deck card balance checksum") + deck.add_argument("--balances", action="store", help="show balances of this deck") card = subparsers.add_parser('card', help='Cards manipulation.') - card.add_argument("--list", action="store", help="list all card transactions of this deck.") - card.add_argument("--export", action="store", nargs=2, help="export all cards of this deck to csv file.") - card.add_argument("--issue", action="store", help="issue cards for this deck.") - card.add_argument("--transfer", action="store", help="send cards to receivers.") - card.add_argument("--burn", action="store", help="burn cards.") + card.add_argument("--list", action="store", help="list all card transactions of this deck") + card.add_argument("--export", action="store", nargs=2, help="export all cards of this deck to csv file") + card.add_argument("--issue", action="store", help="issue cards for this deck") + card.add_argument("--transfer", action="store", help="send cards to receivers") + card.add_argument("--burn", action="store", help="burn cards") return parser.parse_args() From e36599186dd65833b14e0ead4f352028de2fb8d6 Mon Sep 17 00:00:00 2001 From: Esko Date: Thu, 6 Apr 2017 01:01:07 +0200 Subject: [PATCH 3/3] Correct typo --- pacli/__main__.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pacli/__main__.py b/pacli/__main__.py index c943080..ca65880 100644 --- a/pacli/__main__.py +++ b/pacli/__main__.py @@ -636,7 +636,7 @@ def cli(): deck.add_argument("--checksum", action="store", help="verify deck card balance checksum") deck.add_argument("--balances", action="store", help="show balances of this deck") - card = subparsers.add_parser('card', help='Cards manipulation.') + card = subparsers.add_parser('card', help='Card manipulation.') card.add_argument("--list", action="store", help="list all card transactions of this deck") card.add_argument("--export", action="store", nargs=2, help="export all cards of this deck to csv file") card.add_argument("--issue", action="store", help="issue cards for this deck")