Add relevant metadata to issue view in CLI#745
Conversation
cff33a2 to
08c5215
Compare
|
Hi @doi-t thank you for this awesome work! I'll get to looking it over shortly. Quick thought/question though looking at this: is Participants helpful to have here at all? 🤔 cc @mislav @vilmibm @billygriffin if you have thoughts! |
831ccb4 to
1386413
Compare
Yeah, I have the same feeling. Personally, I have never paid attention to This is how |
|
Thanks @doi-t! I'm also in favor of omitting participants - it doesn't seem to add very much in the way of useful context unless we see evidence to the contrary. I'm also not sure how I feel about the truncation after three of each item. This is more of a provocative question than an actual suggestion, but what if we didn't truncate at all? Separately, I'm curious from @mislav and @vilmibm if there are implications to this PR for scripting and machine readability? |
|
@billygriffin Thank you for the review!
Actually, it is not what @ampinsk specifically guided at #663 (comment). I just applied the existing label list implementation which does the truncation for |
|
I'm 👍 removing participants; it doesn't seem very useful in this context.
In my view, I think that a machine-readable |
ebe2e22 to
9169c8d
Compare
|
@mislav Here is the example of no truncation. This PR is ready for code review too! |
vilmibm
left a comment
There was a problem hiding this comment.
I think we all agreed to remove Participants for now but other than that looking good! Thank you!!
Once participants is out I can merge.
|
thank you~ |
|
Hey @doi-t, just wanted to say that we've really appreciated all of your contributions to this project and how thoughtful and open to feedback you've been. Thanks so much! 💖 |
|
@billygriffin |
|
It's absolutely fine and wonderful to hear that you'd like to continue contributing. ❤️ Feel free to ask about upcoming things you're interested in, and we'll happily @-mention you if there are things where we're looking for community contributions, especially in areas you've already contributed. Thanks again! |


This is a part of #663 (
issue view). I will open another PR forpr viewto avoid gigantic PR.(This PR is a nested PR of #667 so that #667 must be merged first.)Thank you for merging the PR! 🎉 😄
@ampinsk
Could you please review the design especially for the case of empty body and lots of metadata?
This is a practical example (
gh issue view 663 --repo cli/cli). Please note that there is noMilestonebecause this issue is not assigned to any milestone.This is an example of an opening issue without metadata settings (
gh issue view 15 --repo doi-t/cli-test).Participantsis set with the author's login account by default and, as far as I know, we cannot avoid it so that we will see it all the time.This is an example of an opening issue with lots of labels.
Labelshas, …at the end of the line in this case because there are more than3labels. Since this issue does not have body content, there is only a new line between metadata and the issue's URL.Regarding to multiple items, while
Milestonecan have only one milestone,Assignees,Labels,ProjectsandParticipantscould have multiple items. I set3as the threshold so far. If there are 4 or more items, each metadata will have, …at the end as shown below (This is just a generated example. Field names will be bolded in the actual output.).