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

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

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.

Summary ID
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.
cd77998c21a5952f9143cbf6ecd5645c160f09d0
chipx86
chipx86
chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-3.x (1c475e4)
Loading...