Merge pull request #84 from RobotsAndPencils/debug

add debug logging
RobotsAndPencils · Oct 16, 2024 · 06a2e5e · 06a2e5e
2 parents f5bb0aa + cc49c4c
commit 06a2e5e
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 9 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -16,7 +16,7 @@ jobs:
           npm install
       - run: |
           npm run all
-  test: # make sure the action works
+  test: # make sure the action works without having to build
     strategy:
       matrix:
         runs-on: [macos-latest, ubuntu-20.04, ubuntu-22.04, ubuntu-latest]
@@ -30,10 +30,6 @@ jobs:
       - name: Checkout Github
         uses: actions/checkout@v3
         if: ${{ !env.ACT }}
-      - name: Install dependencies
-        run: npm install
-      - name: Build Typescript
-        run: npm run all
       - name: Test Action
         uses: ./
         id: secrets

diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/1password.ts b/src/1password.ts
@@ -105,6 +105,10 @@ export class OnePassword {
 
   async signOut(): Promise<void> {
     const env = this.onePasswordEnv
-    await execWithOutput('op', ['signout', '--account', 'github_action', '--forget'], {env})
+    await execWithOutput(
+      'op',
+      ['signout', '--account', 'github_action', '--forget'],
+      {env}
+    )
   }
 }
diff --git a/src/exec.ts b/src/exec.ts
@@ -8,8 +8,9 @@ export async function execWithOutput(
   let out = ''
   let err = ''
 
+  const isDebug = process.env.ACTIONS_STEP_DEBUG === 'true'
   const opt = options ?? {}
-  opt.silent = true // for debugging set this to false to see the output of 1password
+  opt.silent = !isDebug
   opt.listeners = {
     stdout: (data: Buffer) => {
       out += data.toString()