feat(): adjust behavior to more often, and probably more accurately, …

…calculate review sizes and times

package.json

@@ -1,7 +1,7 @@
 {
   "name": "gitmetrix",
   "description": "Helps you find your team-level engineering metrics",
-  "version": "2.1.2",
+  "version": "2.2.0",
   "author": "Mikael Vesavuori",
   "license": "MIT",
   "keywords": [

src/domain/services/GitHubParser.ts

@@ -105,22 +105,15 @@ export class GitHubParser implements Parser {
 
     // PR state: Open
     if (prState === 'open') {
-      if (action === 'ready_for_review') {
-        results.push(this.handleReviewSize(input));
-      } else if (action === 'opened') {
-        results.push(this.resultService.produceResult({ type: 'Opened' }));
-      }
+      results.push(this.handleReviewSize(input));
+      if (action === 'opened') results.push(this.resultService.produceResult({ type: 'Opened' }));
     }
     // PR state: Closed
     else if (prState === 'closed' && action === 'closed') {
       if (mergeState) results.push(this.resultService.produceResult({ type: 'Merged' }));
       results.push(this.resultService.produceResult({ type: 'Closed' }));
       // If this is simply closed, then we have no `merged_at` time
-      if (
-        input['body']['pull_request']['merged_at'] &&
-        input['body']['pull_request']['review_comments'] > 0
-      )
-        results.push(this.handleReviewTime(input));
+      if (input['body']['pull_request']['merged_at']) results.push(this.handleReviewTime(input));
     } else throw new NoPullRequestParsingMatchError();
 
     return results.filter((result: ParsedResult | ParsedResultBasic) => result);

tests/unit/domain/GitHubParser.test.ts

@@ -52,6 +52,16 @@ test.serial('It should parse a "ready for review" event', (t) => {
 
 test.serial('It should parse a "PR opened" event', (t) => {
   const expected: any = [
+    {
+      change: {
+        additions: 1,
+        changedFiles: 1,
+        deletions: 0
+      },
+      repo: 'SOMEORG/SOMEREPO',
+      timestamp: currentTime,
+      type: 'ReviewSize'
+    },
     {
       timestamp: currentTime,
       repo: 'SOMEORG/SOMEREPO',