Merge pull request #172 from christianbundy/exempt-assignees

Add ability to ignore issues with an assignee
probot · Dec 17, 2018 · 5843b68 · 5843b68
2 parents cb88632 + 5d09a9e
commit 5843b68
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 0 deletions.
diff --git a/README.md b/README.md
@@ -36,6 +36,9 @@ exemptProjects: false
 # Set to true to ignore issues in a milestone (defaults to false)
 exemptMilestones: false
 
+# Set to true to ignore issues with an assignee (defaults to false)
+exemptAssignees: false
+
 # Label to use when marking as stale
 staleLabel: wontfix
 

diff --git a/lib/schema.js b/lib/schema.js
@@ -17,6 +17,9 @@ const fields = {
   exemptMilestones: Joi.boolean()
     .description('Set to true to ignore issues in a milestone (defaults to false)'),
 
+  exemptAssignees: Joi.boolean()
+    .description('Set to true to ignore issues with an assignee (defaults to false)'),
+
   staleLabel: Joi.string()
     .description('Label to use when marking as stale'),
 
@@ -43,6 +46,7 @@ const schema = Joi.object().keys({
   exemptLabels: fields.exemptLabels.default(['pinned', 'security']),
   exemptProjects: fields.exemptProjects.default(false),
   exemptMilestones: fields.exemptMilestones.default(false),
+  exemptAssignees: fields.exemptMilestones.default(false),
   staleLabel: fields.staleLabel.default('wontfix'),
   markComment: fields.markComment.default(
     'This issue has been automatically marked as stale because ' +

diff --git a/lib/stale.js b/lib/stale.js
@@ -60,12 +60,14 @@ module.exports = class Stale {
     const exemptLabels = this.getConfigValue(type, 'exemptLabels')
     const exemptProjects = this.getConfigValue(type, 'exemptProjects')
     const exemptMilestones = this.getConfigValue(type, 'exemptMilestones')
+    const exemptAssignees = this.getConfigValue(type, 'exemptAssignees')
     const labels = [staleLabel].concat(exemptLabels)
     const queryParts = labels.map(label => `-label:"${label}"`)
     queryParts.push(Stale.getQueryTypeRestriction(type))
 
     queryParts.push(exemptProjects ? 'no:project' : '')
     queryParts.push(exemptMilestones ? 'no:milestone' : '')
+    queryParts.push(exemptAssignees ? 'no:assignee' : '')
 
     const query = queryParts.join(' ')
     const days = this.getConfigValue(type, 'days') || this.getConfigValue(type, 'daysUntilStale')

diff --git a/test/schema.test.js b/test/schema.test.js
@@ -11,6 +11,8 @@ const validConfigs = [
   [{ exemptProjects: false }],
   [{ exemptMilestones: true }],
   [{ exemptMilestones: false }],
+  [{ exemptAssignees: true }],
+  [{ exemptAssignees: false }],
   [{ staleLabel: 'stale' }],
   [{ markComment: 'stale yo' }],
   [{ markComment: false }],
@@ -33,6 +35,7 @@ const invalidConfigs = [
   [{ daysUntilClose: true }, 'must be a number or false'],
   [{ exemptProjects: 'nope' }, 'must be a boolean'],
   [{ exemptMilestones: 'nope' }, 'must be a boolean'],
+  [{ exemptAssignees: 'nope' }, 'must be a boolean'],
   [{ staleLabel: '' }, 'not allowed to be empty'],
   [{ staleLabel: false }, 'must be a string'],
   [{ staleLabel: ['a', 'b'] }, 'must be a string'],
@@ -58,6 +61,7 @@ describe('schema', () => {
       exemptLabels: ['pinned', 'security'],
       exemptProjects: false,
       exemptMilestones: false,
+      exemptAssignees: false,
       staleLabel: 'wontfix',
       perform: true,
       markComment: 'This issue has been automatically marked as stale because ' +