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)

An issue was opened. 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.