Summary:

Get rid of shadowed --json argument to `rbt status-update`

Review Request #12111 — Created March 7, 2022 and submitted March 8, 2022, 9:10 p.m.

Information
Owner:	david
Repository:	RBTools
Branch:	master
Bugs:
Depends On:
Reviewers
Groups:	rbtools
People:

Description

The rbt status-update command had an existing --json argument, which
was colliding with the new project-wide --json output. This change
updates that command to use the new argument and output objects to
implement the same behavior.

Testing Done

Ran rbt status-update --json and saw it work correctly instead of spew
errors about building the argument parser.

Commits

Summary
	Get rid of shadowed --json argument to `rbt status-update`

Screenshots

Issues

Description	From	Last Updated
Can we generate the list up-front and then set it? Less expensive and a bit more readable.	chipx86	March 8, 2022, 2:27 p.m.

flake8 passed.

JSHint passed.

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

Show all issues

Can we generate the list up-front and then set it? Less expensive and a bit more readable.

david March 7, 2022, 4:33 p.m.

It's not actually any less expensive (the JSON output .append method adds to an existing list-type key), so all it would be doing is adding another variable which has to be kept track of. I suppose it would cut back marginally on function calls but this really isn't a critical path.

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to master (a525550)