From 8d06e6c09483187e4403682d35be4b9267ef3c61 Mon Sep 17 00:00:00 2001 From: Brian Cristante <33549821+brcrista@users.noreply.github.com> Date: Thu, 24 Jun 2021 11:38:43 -0400 Subject: [PATCH] Create check-dist.yml (#227) * Create check-dist.yml * steps.if still needs ${{ }} when the expr has operators * single quotes? * npm install * mess with index.js * add if failure() * Copy the correct version back in from the artifact * Update .github/workflows/check-dist.yml Co-authored-by: Konrad Pabjan * formatting & language * npm ci Co-authored-by: Konrad Pabjan --- .github/workflows/check-dist.yml | 50 ++++++++++++++++++++++++++++++++ .github/workflows/test.yml | 4 +-- 2 files changed, 52 insertions(+), 2 deletions(-) create mode 100644 .github/workflows/check-dist.yml diff --git a/.github/workflows/check-dist.yml b/.github/workflows/check-dist.yml new file mode 100644 index 0000000..3f0f8b5 --- /dev/null +++ b/.github/workflows/check-dist.yml @@ -0,0 +1,50 @@ +# `dist/index.js` is a special file in Actions. +# When you reference an action with `uses:` in a workflow, +# `index.js` is the code that will run. +# For our project, we generate this file through a build process +# from other source files. +# We need to make sure the checked-in `index.js` actually matches what we expect it to be. +name: Check dist/ + +on: + push: + branches: + - main + paths-ignore: + - '**.md' + pull_request: + paths-ignore: + - '**.md' + workflow_dispatch: + +jobs: + check-dist: + runs-on: ubuntu-latest + + steps: + - uses: actions/checkout@v2 + + - name: Set Node.js 12.x + uses: actions/setup-node@v1 + with: + node-version: 12.x + + - name: Install dependencies + run: npm ci + + - name: Move the committed index.js file + run: mv dist/index.js /tmp + + - name: Rebuild the index.js file + run: npm run release + + - name: Compare the expected and actual index.js files + run: git diff --ignore-all-space dist/index.js /tmp/index.js + id: diff + + # If index.js was different than expected, upload the expected version as an artifact + - uses: actions/upload-artifact@v2 + if: ${{ failure() && steps.diff.conclusion == 'failure' }} + with: + name: index.js + path: dist/index.js diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 0233186..1d9caa7 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -30,8 +30,8 @@ jobs: with: node-version: 12.x - - name: npm install - run: npm install + - name: Install dependencies + run: npm ci - name: Compile run: npm run build