Provide all metadata fields in scripted pr view #1532
Description
Describe the feature or problem you’d like to solve
I am trying to automate doing something with the PR just created from the current branch. gh pr view will print out the PR info including the URL, which would be fine, except that if your redirect stdout it drops this information. (I understand having a command behave differently in interactive vs scripted contexts, but this seems like a pretty surprising behavior).
Proposed solution
All information in the interactive view should also be present in the scripted view. In particular gh pr view should include the PR number and URL (and/or include the owner and repo name as separate fields).
How will it benefit CLI and its users?
This enables gh to be used with other automation that operates on PRs.
Additional context
Current behavior
gcmn@ws 2020-08-14 10:56 ~/git/iree$ gh pr view
Merge main -> google
Open • GMNGeoffrey wants to merge 14 commits into google from main-to-google
Labels: cla: yes
• e15008ca Standardize E2E module names. (#2872)
• e5d0f61e Add support for mhlo.log_plus_one on all the backends. (2873)
• 177925c4 Mark unanalyzable XLA target as manual (#2881)
• 8204a7a8 Merge google -> main #2877
• 45d5f5a1 Avoid confusing error message in TF tests (#2879)
• 064dd2b7 Merge pull request #2742 from google/benvanik-vm64-flags
• c2e59d2e Workaround for issue in #2878.
• 99ed353d Reworking type conversion to support VM target options.
• a8ef9dc3 Supporting new VM TargetOptions type conversion in std->vm.
• 0a473431 Plumbing VM TargetOptions through conversion passes.
• 87c96f26 Adding IREE::VM::TargetOptions and --iree-vm-target-* flags.
• 2ff8b399 Removing unused TypeConversionUtils.
• 3546d64c Merge google -> main (#2876)
• d13546b2 Remove uses of kind on derived mlir Attribute and Type classes
(#2861)
View this pull request on GitHub: https://github.com/google/iree/pull/2887
gcmn@ws 2020-08-14 10:56 ~/git/iree$ gh pr view > /tmp/pr.txt && cat /tmp/pr.txt
title: Merge main -> google
state: OPEN
author: GMNGeoffrey
labels: cla: yes
assignees:
reviewers:
projects:
milestone:
--
* e15008ca Standardize E2E module names. (#2872)
* e5d0f61e Add support for mhlo.log_plus_one on all the backends. (2873)
* 177925c4 Mark unanalyzable XLA target as manual (#2881)
* 8204a7a8 Merge google -> main #2877
* 45d5f5a1 Avoid confusing error message in TF tests (#2879)
* 064dd2b7 Merge pull request #2742 from google/benvanik-vm64-flags
* c2e59d2e Workaround for issue in #2878.
* 99ed353d Reworking type conversion to support VM target options.
* a8ef9dc3 Supporting new VM TargetOptions type conversion in std->vm.
* 0a473431 Plumbing VM TargetOptions through conversion passes.
* 87c96f26 Adding IREE::VM::TargetOptions and --iree-vm-target-* flags.
* 2ff8b399 Removing unused TypeConversionUtils.
* 3546d64c Merge google -> main (#2876)
* d13546b2 Remove uses of kind on derived mlir Attribute and Type classes (#2861)```
Proposed behavior
gcmn@ws 2020-08-14 10:56 ~/git/iree$ gh pr view
Merge main -> google
Open • GMNGeoffrey wants to merge 14 commits into google from main-to-google
Labels: cla: yes
• e15008ca Standardize E2E module names. (#2872)
• e5d0f61e Add support for mhlo.log_plus_one on all the backends. (2873)
• 177925c4 Mark unanalyzable XLA target as manual (#2881)
• 8204a7a8 Merge google -> main #2877
• 45d5f5a1 Avoid confusing error message in TF tests (#2879)
• 064dd2b7 Merge pull request #2742 from google/benvanik-vm64-flags
• c2e59d2e Workaround for issue in #2878.
• 99ed353d Reworking type conversion to support VM target options.
• a8ef9dc3 Supporting new VM TargetOptions type conversion in std->vm.
• 0a473431 Plumbing VM TargetOptions through conversion passes.
• 87c96f26 Adding IREE::VM::TargetOptions and --iree-vm-target-* flags.
• 2ff8b399 Removing unused TypeConversionUtils.
• 3546d64c Merge google -> main (#2876)
• d13546b2 Remove uses of kind on derived mlir Attribute and Type classes
(#2861)
View this pull request on GitHub: https://github.com/google/iree/pull/2887
gcmn@ws 2020-08-14 10:56 ~/git/iree$ gh pr view > /tmp/pr.txt && cat /tmp/pr.txt
title: Merge main -> google
state: OPEN
author: GMNGeoffrey
labels: cla: yes
number: 2887
commit_count: 14
url: https://github.com/google/iree/pull/2887
base_repo_owner: google
base_repo: iree
base_branch: google
pr_repo_owner: GMNGeoffrey
pr_repo: iree
pr_branch: main-to-google
assignees:
reviewers:
projects:
milestone:
--
* e15008ca Standardize E2E module names. (#2872)
* e5d0f61e Add support for mhlo.log_plus_one on all the backends. (2873)
* 177925c4 Mark unanalyzable XLA target as manual (#2881)
* 8204a7a8 Merge google -> main #2877
* 45d5f5a1 Avoid confusing error message in TF tests (#2879)
* 064dd2b7 Merge pull request #2742 from google/benvanik-vm64-flags
* c2e59d2e Workaround for issue in #2878.
* 99ed353d Reworking type conversion to support VM target options.
* a8ef9dc3 Supporting new VM TargetOptions type conversion in std->vm.
* 0a473431 Plumbing VM TargetOptions through conversion passes.
* 87c96f26 Adding IREE::VM::TargetOptions and --iree-vm-target-* flags.
* 2ff8b399 Removing unused TypeConversionUtils.
* 3546d64c Merge google -> main (#2876)
* d13546b2 Remove uses of kind on derived mlir Attribute and Type classes (#2861)```