Summary

Add pretty print short option to api_get

Review Request #9794 — Created March 16, 2018 and updated Jan. 3, 2019, 12:19 p.m.

Information

Owner

solarmist

Repository

RBTools

Branch

master

Bugs

Depends On

Commit

49d0f1b...

Reviewers

Groups

rbtools

People

chipx86, david

Description

Add pretty print short option to api_get

Testing Done

Tested in python 2.7

./bin/rbt-2.7 api-get -pp webhooks
{
    "err": {
        "code": 101,
        "msg": "You don't have permission for this"
    },
    "stat": "fail"
}

Issues

Description	From	Last Updated
I've been thinking about this, and this is the only place where we have a two-character "short" arg. I think …	david	Jan. 3, 2019, 12:19 p.m.

flake8 passed.

JSHint passed.

Change Summary:

Ping

People:

+	chipx86
+	david

Joshua,

We're currently frozen for the release of RBTools 1.0. We'll get this in as soon as things unlock after the release is done.

solarmist Jan. 2, 2019, 11:19 a.m.
```
Can you pull this in now?
```

Fix it!

rbtools/commands/api_get.py (Diff revision 1)

Show all issues

I've been thinking about this, and this is the only place where we have a two-character "short" arg. I think if we want to do this at all, we really should stay consistent and make it -p or -P.