Merge pull request #11 from kyagi/refactoring-20220204-1

Conditional logging when run on github actions

Author: kyagi

dist/index.js

@@ -23670,19 +23670,6 @@ function wrappy (fn, cb) {
 /***/ 6092:
 /***/ ((module, __unused_webpack_exports, __nccwpck_require__) => {
 
-// const {Octokit} = require("@octokit/core")
-// let octokit = new Octokit({auth: process.env.shamshir_pat})
-//
-// const core = require('@actions/core')
-// const github = require('@actions/github')
-// const GITHUB_TOKEN = core.getInput('GITHUB_TOKEN')
-// if (!process.env.shamshir_pat) {
-//     octokit = github.getOctokit(GITHUB_TOKEN)
-// }
-// const {context = {}} = github
-// const {pull_request} = context.payload
-//
-
 const { Octokit } = __nccwpck_require__(6762)
 const core = __nccwpck_require__(2186)
 const github = __nccwpck_require__(5438)
@@ -23711,6 +23698,7 @@ module.exports = class Shamshir {
             this._quorum = core.getInput('quorum')
             this._mode = false
         } else {
+            // TODO: error-handing, TBD
             // Neither stand-alone nor github actions
         }
     }
@@ -23753,11 +23741,15 @@ module.exports = class Shamshir {
                 level: 'error', message: `${error}`, owner: owner, repo: repo, function: 'main', mode: mode })
         } finally {
             // TODO: fix this
-            const fs = __nccwpck_require__(7147)
-            fs.readFile('combined.log', 'utf-8', (err, files) => {
-                if (err) { throw err; }
-                core.setOutput('log', files)
-            })
+            if (GITHUB_TOKEN) {
+                const fs = __nccwpck_require__(7147)
+                fs.readFile('combined.log', 'utf-8', (err, files) => {
+                    if (err) {
+                        throw err;
+                    }
+                    core.setOutput('log', files)
+                })
+            }
             logger.log({level: 'info', message: 'Shamshir finished.', owner: owner, repo: repo, mode: mode })
         }
     }

src/shamshir-stand-alone.js

@@ -1,11 +1,12 @@
-const Shamshir = require("./shamshir.js")
 const { argv } = require('./yargs.js')
 const { owner, repo, label, quorum, check } = argv
 
 let mode = "live"
 if (check === true) {
     mode = "dry-run"
 }
+
+const Shamshir = require("./shamshir.js")
 const shamshir = new Shamshir(owner, repo, label, quorum, mode)
 
 // TODO: fix or use top level await

src/shamshir.js

@@ -1,16 +1,3 @@
-// const {Octokit} = require("@octokit/core")
-// let octokit = new Octokit({auth: process.env.shamshir_pat})
-//
-// const core = require('@actions/core')
-// const github = require('@actions/github')
-// const GITHUB_TOKEN = core.getInput('GITHUB_TOKEN')
-// if (!process.env.shamshir_pat) {
-//     octokit = github.getOctokit(GITHUB_TOKEN)
-// }
-// const {context = {}} = github
-// const {pull_request} = context.payload
-//
-
 const { Octokit } = require("@octokit/core")
 const core = require('@actions/core')
 const github = require('@actions/github')
@@ -39,6 +26,7 @@ module.exports = class Shamshir {
             this._quorum = core.getInput('quorum')
             this._mode = false
         } else {
+            // TODO: error-handing, TBD
             // Neither stand-alone nor github actions
         }
     }
@@ -81,11 +69,15 @@ module.exports = class Shamshir {
                 level: 'error', message: `${error}`, owner: owner, repo: repo, function: 'main', mode: mode })
         } finally {
             // TODO: fix this
-            const fs = require('fs')
-            fs.readFile('combined.log', 'utf-8', (err, files) => {
-                if (err) { throw err; }
-                core.setOutput('log', files)
-            })
+            if (GITHUB_TOKEN) {
+                const fs = require('fs')
+                fs.readFile('combined.log', 'utf-8', (err, files) => {
+                    if (err) {
+                        throw err;
+                    }
+                    core.setOutput('log', files)
+                })
+            }
             logger.log({level: 'info', message: 'Shamshir finished.', owner: owner, repo: repo, mode: mode })
         }
     }