From 7f62911d6564c468d53675b4c0ff018cbf86e6ca Mon Sep 17 00:00:00 2001 From: Joey Parrish Date: Tue, 5 Nov 2024 11:28:05 -0800 Subject: [PATCH] ci: Fix workflow events This was found by auditing workflows based on research published here: https://github.com/joeyparrish/workflow-cheat-sheet The research was prompted by a workflow bug in Shaka Streamer --- .github/workflows/release-please.yml | 9 ++++++++- .github/workflows/test.yaml | 2 +- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/.github/workflows/release-please.yml b/.github/workflows/release-please.yml index 7d5bc514..ad8c5d6a 100644 --- a/.github/workflows/release-please.yml +++ b/.github/workflows/release-please.yml @@ -17,21 +17,28 @@ jobs: # The logic below handles npm publication. Each step is conditional on a # release having been created by someone merging the release PR. - - uses: actions/checkout@v2 + - uses: actions/checkout@v4 + with: + ref: refs/tags/${{ steps.release.outputs.tag_name }} if: ${{ steps.release.outputs.release_created }} + - uses: actions/setup-node@v1 with: node-version: 12 registry-url: 'https://registry.npmjs.org' if: ${{ steps.release.outputs.release_created }} + - run: npm ci if: ${{ steps.release.outputs.release_created }} + - run: npm publish env: NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}} if: ${{ steps.release.outputs.release_created }} + - run: npm pack if: ${{ steps.release.outputs.release_created }} + - uses: svenstaro/upload-release-action@483c1e56f95e88835747b1c7c60581215016cbf2 with: repo_token: ${{ secrets.GITHUB_TOKEN }} diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml index 35cf5c44..bfc177c5 100644 --- a/.github/workflows/test.yaml +++ b/.github/workflows/test.yaml @@ -17,7 +17,7 @@ jobs: - name: Checkout code uses: actions/checkout@v2 with: - ref: ${{ github.event.inputs.ref || github.ref }} + ref: ${{ inputs.ref || github.ref }} - name: Test run: |