Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add PR reviews metrics. #1567

Merged
merged 1 commit into from Apr 9, 2018
Merged

Conversation

@grokys
Copy link
Contributor

@grokys grokys commented Mar 27, 2018

Adds two new metrics:

  • NumberOfPRReviewDiffViewInlineCommentStartReview: incremented when the "Start Review" button is clicked from an inline comment
  • NumberOfPRReviewPosts: incremented when a PR review is submitted

Depends on #1562
Part of #1491

Adds two new metrics:

- `NumberOfPRReviewDiffViewInlineCommentStartReview`: incremented when the "Start Review" button is clicked from an inline comment
- `NumberOfPRReviewPosts`: incremented when a PR review is submitted
@grokys grokys mentioned this pull request Mar 27, 2018
17 of 17 tasks complete
@meaghanlewis
Copy link
Contributor

@meaghanlewis meaghanlewis commented Mar 27, 2018

These metrics look good to me! Tested on VS2015 & VS2017

@sguthals
Copy link
Contributor

@sguthals sguthals commented Mar 28, 2018

The first time I started a review from a diff and submitted that review it did not get recorded. I've tried twice since then and it worked. I'm going to try re-starting my experimental instance to see if the first one works when I re-start.

@sguthals
Copy link
Contributor

@sguthals sguthals commented Mar 29, 2018

Tests:
2015 Dogfood:

  • responding to a comment to start a review did not cause the comment to remain open
  • all metrics were updated

2015 Experimental:

  • responding to a comment to start a review did not cause the comment to remain open
  • all metrics were updated
  • This time I submitted as comment only and it increased numberOfPRReviewPosts - which I believe is intended behavior, I just wanted to make sure 👍

2017 Experimental:

  • responding to a comment to start a review did not cause the comment to remain open
  • all metrics were updated
  • This time I submitted as approved and it still worked 👍
Copy link
Contributor

@sguthals sguthals left a comment

Looks good! Thanks for fixing that one bug. I have not been able to repro the first time that my metrics weren't updated, but I'm going to assume that was just funky with me for now.

@hubot hubot requested a deployment to production Mar 29, 2018 Abandoned
@jcansdale jcansdale changed the base branch from feature/pr-review-authoring to feature/pr-reviews-master Apr 9, 2018
@jcansdale jcansdale force-pushed the feature/pr-review-metrics branch from 3ba7773 to 4e69c17 Apr 9, 2018
@jcansdale jcansdale merged commit b4cf1d0 into feature/pr-reviews-master Apr 9, 2018
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/appveyor/branch Waiting for AppVeyor build to complete
Details
continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
@jcansdale jcansdale deleted the feature/pr-review-metrics branch Apr 9, 2018
@meaghanlewis meaghanlewis added this to Done in 2.4.4 Apr 13, 2018
@meaghanlewis meaghanlewis removed this from Done in 2.4.4 Apr 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.