Make it compatible with Probot 12

index.js

@@ -2,7 +2,7 @@ const Configuration = require('./lib/configuration')
 const HaltedError = require('./lib/errors/halted')
 
 module.exports = robot => {
-  robot.on('*', async context => {
+  robot.webhooks.onAny(async context => {
     const config = await Configuration.load(context, '.github/probot.js')
     return config.execute().catch(err => {
       if (err instanceof HaltedError) {

lib/configuration.js

@@ -41,7 +41,7 @@ module.exports = class Configuration {
     const options = path
       ? this.context.repo(url(path, this.source))
       : this.source
-    return this.context.github.repos.getContent(options).then(res =>
+    return this.context.octokit.repos.getContent(options).then(res =>
       Buffer.from(res.data.content, 'base64').toString()
     )
   }

lib/plugins/filter.js

@@ -14,7 +14,7 @@ module.exports = class Filter extends Plugin {
   on (context, ...events) {
     const res = events.find(e => {
       const [name, action] = e.split('.')
-      return name === context.event && (!action || action === context.payload.action)
+      return name === context.name && (!action || action === context.payload.action)
     })
 
     return res

lib/plugins/issues.js

@@ -4,48 +4,48 @@ const Plugin = require('../plugin')
 module.exports = class Issues extends Plugin {
   comment (context, content) {
     const template = handlebars.compile(content)(context.payload)
-    return context.github.issues.createComment(context.issue({body: template}))
+    return context.octokit.issues.createComment(context.issue({body: template}))
   }
 
   assign (context, ...assignees) {
-    return context.github.issues.addAssigneesToIssue(context.issue({assignees}))
+    return context.octokit.issues.addAssigneesToIssue(context.issue({assignees}))
   }
 
   unassign (context, ...assignees) {
-    return context.github.issues.removeAssigneesFromIssue(context.issue({body: {assignees}}))
+    return context.octokit.issues.removeAssigneesFromIssue(context.issue({body: {assignees}}))
   }
 
   label (context, ...labels) {
-    return context.github.issues.addLabels(context.issue({labels}))
+    return context.octokit.issues.addLabels(context.issue({labels}))
   }
 
   unlabel (context, ...labels) {
     return labels.map(label => {
-      return context.github.issues.removeLabel(
+      return context.octokit.issues.removeLabel(
         context.issue({name: label})
       )
     })
   }
 
   lock (context) {
-    return context.github.issues.lock(context.issue({}))
+    return context.octokit.issues.lock(context.issue({}))
   }
 
   unlock (context) {
-    return context.github.issues.unlock(context.issue({}))
+    return context.octokit.issues.unlock(context.issue({}))
   }
 
   open (context) {
-    return context.github.issues.edit(context.issue({state: 'open'}))
+    return context.octokit.issues.update(context.issue({state: 'open'}))
   }
 
   close (context) {
-    return context.github.issues.edit(context.issue({state: 'closed'}))
+    return context.octokit.issues.update(context.issue({state: 'closed'}))
   }
 
   deleteComment (context) {
     const comment = context.payload.comment
-    const github = context.github
+    const github = context.octokit
 
     const deleteFunction =
       (comment.pull_request_review_id && github.pullRequests.deleteComment) ||
@@ -60,7 +60,7 @@ module.exports = class Issues extends Plugin {
       const titleTemplate = handlebars.compile(content.title)(context.payload)
       const bodyTemplate = handlebars.compile(body)(context.payload)
 
-      return context.github.issues.create(context.repo({
+      return context.octokit.issues.create(context.repo({
         title: titleTemplate,
         body: bodyTemplate,
         assignees: content.assignees,

lib/plugins/projects.js

@@ -11,7 +11,7 @@ module.exports = class Projects extends Plugin {
   createCard (context, content) {
     return Promise.resolve().then(() => {
       // Get all projects for repo
-      Projects.getRepoProjects(context)
+      Projects.listForRepo(context)
       // Then get the projectID for desired project
         .then(response => Projects.getID(response, content.project))
         // Then get the columns for that project
@@ -20,7 +20,7 @@ module.exports = class Projects extends Plugin {
         .then(projectColumns => Projects.getID(projectColumns, content.column))
         // Then create the card
         .then(columnID => {
-          context.github.projects.createProjectCard({
+          context.octokit.projects.createProjectCard({
             column_id: columnID,
             content_id: context.payload.issue.id,
             content_type: 'Issue'
@@ -35,8 +35,8 @@ module.exports = class Projects extends Plugin {
    * @param context
    * @returns {*}
    */
-  static getRepoProjects (context) {
-    return context.github.projects.getRepoProjects({
+  static listForRepo (context) {
+    return context.octokit.projects.listForRepo({
       owner: context.payload.repository.owner.login,
       repo: context.payload.repository.name
     })
@@ -64,6 +64,6 @@ module.exports = class Projects extends Plugin {
    * @returns {*}
    */
   static getProjectColumns (context, projectId) {
-    return context.github.projects.getProjectColumns({projectId})
+    return context.octokit.projects.getProjectColumns({projectId})
   }
 }

package.json

@@ -11,7 +11,7 @@
   },
   "dependencies": {
     "handlebars": "^4.0.6",
-    "probot": "^6.1.0"
+    "probot": "^12.1.0"
   },
   "engines": {
     "node": ">= 7.7.0",

test/configuration.js

@@ -36,7 +36,7 @@ describe('Configuration', () => {
 
     it('includes from the repo', () => {
       config.include('foo.js')
-      expect(context.github.repos.getContent).toHaveBeenCalledWith({
+      expect(context.octokit.repos.getContent).toHaveBeenCalledWith({
         path: 'foo.js'
       })
     })
@@ -47,7 +47,7 @@ describe('Configuration', () => {
 
     it('includes from another repository', () => {
       config.include('atom/configs:foo.js#branch')
-      expect(context.github.repos.getContent).toHaveBeenCalledWith({
+      expect(context.octokit.repos.getContent).toHaveBeenCalledWith({
         owner: 'atom',
         repo: 'configs',
         path: 'foo.js',

test/index.js

@@ -68,7 +68,7 @@ describe('app', () => {
           .filter((e) => e.payload.label.name == "bug")
           .close();
         `, context)
-      expect(github.issues.edit).toHaveBeenCalled()
+      expect(github.issues.update).toHaveBeenCalled()
     })
 
     it('does not call action when conditions do not match', async () => {
@@ -77,7 +77,7 @@ describe('app', () => {
           .filter((e) => e.payload.label.name == "foobar")
           .close();
         `, context)
-      expect(github.issues.edit).toHaveBeenCalledTimes(0)
+      expect(github.issues.update).toHaveBeenCalledTimes(0)
     })
   })
 

test/plugins/filter.js

@@ -56,15 +56,15 @@ describe('filter plugin', () => {
   describe('on', () => {
     describe('matching only the event name', () => {
       it('matches on a single event', () => {
-        context.event = 'issues'
+        context.name = 'issues'
 
         return filter.on(context, 'issues').then(result => {
           expect(result).toEqual('issues')
         })
       })
 
       it('fails to match on a single event', () => {
-        context.event = 'issues'
+        context.name = 'issues'
 
         return filter.on(context, 'foo').catch(err => {
           expect(err).toBeInstanceOf(HaltedError)
@@ -73,15 +73,15 @@ describe('filter plugin', () => {
       })
 
       it('matches any of the event names', () => {
-        context.event = 'foo'
+        context.name = 'foo'
 
         return filter.on(context, 'issues', 'foo').then(result => {
           expect(result).toEqual('foo')
         })
       })
 
       it('fails to match if none of the event names match', () => {
-        context.event = 'bar'
+        context.name = 'bar'
 
         return filter.on(context, 'issues', 'foo').catch(err => {
           expect(err).toBeInstanceOf(HaltedError)
@@ -92,7 +92,7 @@ describe('filter plugin', () => {
 
     describe('matching the event and action', () => {
       it('matches on a single event', () => {
-        context.event = 'issues'
+        context.name = 'issues'
         context.payload = {action: 'opened'}
 
         return filter.on(context, 'issues.opened').then(result => {
@@ -101,7 +101,7 @@ describe('filter plugin', () => {
       })
 
       it('fails to match on a single event', () => {
-        context.event = 'issues'
+        context.name = 'issues'
         context.payload = {action: 'foo'}
 
         return filter.on(context, 'issues.opened').catch(err => {
@@ -111,7 +111,7 @@ describe('filter plugin', () => {
       })
 
       it('matches any of the event descriptors', () => {
-        context.event = 'issues'
+        context.name = 'issues'
         context.payload = {action: 'closed'}
 
         return filter.on(context, 'issues.opened', 'issues.closed').then(result => {
@@ -120,7 +120,7 @@ describe('filter plugin', () => {
       })
 
       it('fails to match if none of the event descriptors match', () => {
-        context.event = 'issues'
+        context.name = 'issues'
         context.payload = {action: 'foo'}
 
         return filter.on(context, 'issues.opened', 'issues.closed').catch(err => {