Merge pull request #1 from adamzolyak/refactorAuth

Update Octokit to use new auth method
TinkurLab · Feb 21, 2019 · d999b30 · d999b30
2 parents ed4948b + 50d620f
commit d999b30
Show file tree

Hide file tree

Showing 5 changed files with 31 additions and 40 deletions.
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -27,7 +27,7 @@ Test Status: [![CircleCI](https://circleci.com/gh/adamzolyak/issue-labeler-actio
 ```bash
 export GITHUB_TOKEN="12345"
 export GITHUB_REPOSITORY="adamzolyak/actions-playground"
-export GITHUB_EVENT_PATH="/tests/fixtures/actionTrigger.json"
+export GITHUB_EVENT_PATH="/issue-labeler-action/tests/fixtures/actionTrigger.json"
 
 node index.js
 ```

diff --git a/app.js b/app.js
@@ -4,12 +4,9 @@ const helpers = require('./helpers')
 
 //require octokit rest.js
 //more info at https://github.com/octokit/rest.js
-const octokit = require('@octokit/rest')()
-
-//set octokit auth to action's GITHUB_TOKEN env variable
-octokit.authenticate({
-  type: 'app',
-  token: process.env.GITHUB_TOKEN
+const Octokit = require('@octokit/rest')
+const octokit = new Octokit({
+  auth: `token ${process.env.GITHUB_TOKEN}`
 })
 
 //set eventOwner and eventRepo based on action's env variables

diff --git a/dev.example b/dev.example
@@ -1,6 +1,6 @@
 
 export GITHUB_TOKEN="12345"
 export GITHUB_REPOSITORY="adamzolyak/actions-playground"
-export GITHUB_EVENT_PATH="/action-checklistchecker/tests/fixtures/actionTrigger.json"
+export GITHUB_EVENT_PATH="/issue-labeler-action/tests/fixtures/actionTrigger.json"
 
 node index.js
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -26,7 +26,7 @@
   },
   "homepage": "https://www.tinkurlab.com",
   "dependencies": {
-    "@octokit/rest": "^16.2.0"
+    "@octokit/rest": "^16.16.0"
   },
   "devDependencies": {
     "jest": "^23.6.0"