Enhance the JSON output for rbt status and clean up some non-standard code.

Review Request #12177 — Created March 18, 2022 and submitted — Latest diff uploaded

Information

RBTools
release-3.x

Reviewers

This enhances the results of rbt status --json to show additional
information useful for automation.

The results are now in a review_requests key, instead of summaries,
which better identifies the type of results and allows for enhancement
in the future.

The review request ID (review_request_id) and URL
(review_request_url) are now provided, instead of just the ID.

The results also now include has_draft, shipit_count,
open_issue_count, approved, and approval_failure fields, which are
more useful for tools that may want to wrap this command.

It also cleans up some of the code to use more standard terminology
("review request" instead of "request" or "review"), and to fix
documentation and spacing.

Tested rbt status --json and saw the revised fields.

Diff Revision 3

This is not the most recent revision of the diff. The latest diff is revision 4. See what's changed.

orig
1
2
3
4

Commits

First Last Summary ID Author
Enhance the JSON output for rbt status and clean up some non-standard code.
This enhances the results of `rbt status --json` to show additional information useful for automation. The results are now in a `review_requests` key, instead of `summaries`, which better identifies the type of results and allows for enhancement in the future. The review request ID (`review_request_id`) and URL (`review_request_url`) are now provided, instead of just the ID. The results also now include `has_draft`, `shipit_count`, `open_issue_count`, `approved`, and `approval_failure` fields, which are more useful for tools that may want to wrap this command. It also cleans up some of the code to use more standard terminology ("review request" instead of "request" or "review"), and to fix documentation and spacing.
7c00910c7e5088ed77839a61d6568a8f0b381b02 Christian Hammond
rbtools/commands/status.py
Loading...