feat: add an SLO for PRs

labels.json

@@ -75,6 +75,11 @@
       "description": "Required feature/issue must be fixed prior to next release.",
       "color": "ffa03e"
     },
+    {
+      "name": "needs work",
+      "description": "This is a pull request that needs a little love.",
+      "color": "1a3b70"
+    },
     {
       "name": "type: docs",
       "description": "Improvement to the documentation for an API.",
@@ -123,12 +128,12 @@
     {
       "name": "help wanted",
       "description": "We'd love to have community involvement on this issue.",
-      "color": "FFFFFF"
+      "color": "3f5e3d"
     },
     {
       "name": "good first issue",
       "description": "This issue is a good place to started contributing to this repository.",
-      "color": "FFFFFF"
+      "color": "520b77"
     }
   ]
 }

src/issue.ts

@@ -19,7 +19,7 @@ import {Flags, Issue, IssueResult, Repo} from './types';
 import {octo, repos} from './util';
 
 import Table = require('cli-table');
-import {isTriaged, isOutOfSLO, hasLabel, isApi, isPullRequest, hoursOld, getApi} from './slo';
+import {isTriaged, isOutOfSLO, hasLabel, isPullRequest, hoursOld, getApi} from './slo';
 const truncate = require('truncate');
 const CSV = require('csv-string');
 

src/slo.ts

@@ -310,8 +310,15 @@ export function hoursOld(date: string) {
  * @param i Issue to analyze
  */
 export function isOutOfSLO(i: Issue) {
-  // Pull requests don't count.
+  // Pull requests must be merged within a week, unless they have a
+  // 'needs work' label.  After 90 days, it should just be resolved.
   if (isPullRequest(i)) {
+    if (daysOld(i.created_at) > 90) {
+      return true;
+    }
+    if (daysOld(i.created_at) > 7 && !hasLabel(i, 'needs work')) {
+      return true;
+    }
     return false;
   }