chore: split Github Actions workflows

.github/workflows/build.yml

@@ -0,0 +1,40 @@
+name: Build
+on:
+  workflow_call:
+    inputs:
+      os:
+        required: true
+        type: string
+      bundles:
+        required: false
+        type: string
+env:
+  BUNDLES: ${{ inputs.bundles }}
+jobs:
+  build:
+    name: Build
+    runs-on: ${{ inputs.os }}
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+        with:
+          fetch-depth: 0
+      - name: Setup Node.js
+        uses: actions/setup-node@v2
+        with:
+          node-version: '14'
+      - name: Install dependencies
+        run: npm ci
+      - name: Build
+        run: npm run build
+      - name: Archive artifacts
+        uses: actions/upload-artifact@v3
+        with:
+          name: dist-${{ inputs.os }}
+          path: dist
+      - name: Archive npm failure logs
+        uses: actions/upload-artifact@v2
+        if: failure()
+        with:
+          name: npm-logs
+          path: ~/.npm/_logs

.github/workflows/ci-build.yml

@@ -0,0 +1,28 @@
+name: CI Build
+on:
+  push:
+    branches:
+      - 'master'
+      - 'test'
+jobs:
+  build:
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest]
+    uses: ./.github/workflows/build.yml
+    with:
+      os: ${{ matrix.os }}
+  lint:
+    uses: ./.github/workflows/lint.yml
+  test:
+    needs: build
+    uses: ./.github/workflows/test.yml
+  coverage:
+    uses: ./.github/workflows/coverage.yml
+  performance:
+    needs: build
+    uses: ./.github/workflows/performance.yml
+    with:
+      os: ubuntu-latest
+  docs:
+    uses: ./.github/workflows/docs.yml

.github/workflows/check.yml → .github/workflows/coverage.yml

@@ -1,12 +1,9 @@
-name: Check
-on: [push, pull_request]
+name: Test Coverage
+on: workflow_call
 jobs:
-  check:
-    name: Check
-    strategy:
-      matrix:
-        os: [ubuntu-latest, macos-latest]
-    runs-on: ${{ matrix.os }}
+  coverage:
+    name: Test Coverage
+    runs-on: ubuntu-latest
     steps:
       - name: Checkout
         uses: actions/checkout@v2
@@ -16,23 +13,14 @@ jobs:
         uses: actions/setup-node@v2
         with:
           node-version: '14'
-      - name: Build
-        run: |
-          npm ci
-          npm run build
+      - name: Install dependencies
+        run: npm ci
       - name: Test
-        run: |
-          npm run lint
-          TZ=Etc/GMT npm run test
-          TZ=Asia/Shanghai npm run test
-          TZ=America/New_York npm run test
+        run: npm run test:coverage
       - name: Coverage
         uses: coverallsapp/github-action@v1.1.2
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
-      - name: Performance
-        run: |
-          npm run perf:diff
       - name: Archive npm failure logs
         uses: actions/upload-artifact@v2
         if: failure()

.github/workflows/docs.yml

@@ -1,8 +1,5 @@
 name: Docs
-on:
-  push:
-    branches:
-      - master
+on: [workflow_dispatch, workflow_call]
 jobs:
   docs:
     name: Docs
@@ -19,8 +16,9 @@ jobs:
       - name: Build
         run: |
           npm ci
-          npm run build
+          npm run build:docs
       - name: Publish
+        if: github.ref == 'refs/heads/master'
         uses: JamesIves/github-pages-deploy-action@4.1.4
         with: 
           branch: gh-pages

.github/workflows/lint.yml

@@ -0,0 +1,25 @@
+name: Lint
+on: [workflow_dispatch, workflow_call]
+jobs:
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+        with:
+          fetch-depth: 0
+      - name: Setup Node.js
+        uses: actions/setup-node@v2
+        with:
+          node-version: '14'
+      - name: Install dependencies
+        run: npm ci
+      - name: Lint
+        run: npm run lint
+      - name: Archive npm failure logs
+        uses: actions/upload-artifact@v2
+        if: failure()
+        with:
+          name: npm-logs
+          path: ~/.npm/_logs

.github/workflows/performance.yml

@@ -0,0 +1,33 @@
+name: Performance Check
+on:
+  workflow_call:
+    inputs:
+      os:
+        required: true
+        type: string
+jobs:
+  performance:
+    name: Performance
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+        with:
+          fetch-depth: 0
+      - name: Setup Node.js
+        uses: actions/setup-node@v2
+        with:
+          node-version: '14'
+      - name: Download artifacts
+        uses: actions/download-artifact@v3
+        with:
+          name: dist-${{ inputs.os }}
+          path: dist
+      - name: Check Performance
+        run: npm run perf:diff
+      - name: Archive npm failure logs
+        uses: actions/upload-artifact@v2
+        if: failure()
+        with:
+          name: npm-logs
+          path: ~/.npm/_logs

.github/workflows/pr-build.yml

@@ -0,0 +1,22 @@
+name: PR Build
+on: pull_request
+jobs:
+  build:
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest]
+    uses: ./.github/workflows/build.yml
+    with:
+      os: ${{ matrix.os }}
+  lint:
+    uses: ./.github/workflows/lint.yml
+  test:
+    needs: build
+    uses: ./.github/workflows/test.yml
+  coverage:
+    uses: ./.github/workflows/coverage.yml
+  performance:
+    needs: build
+    uses: ./.github/workflows/performance.yml
+    with:
+      os: ubuntu-latest

.github/workflows/pr.yml → .github/workflows/pr-check.yml

@@ -1,15 +1,12 @@
-name: "PR Title Checker"
-
+name: PR Check
 on:
   pull_request_target:
     types:
       - opened
       - edited
-      - synchronize
-
 jobs:
-  main:
-    name: Validate PR title
+  title:
+    name: Check PR title
     runs-on: ubuntu-latest
     steps:
       - uses: amannn/action-semantic-pull-request@v5

.github/workflows/release.yml

@@ -1,8 +1,19 @@
 name: Release
 on: workflow_dispatch
 jobs:
+  build:
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest]
+    uses: ./.github/workflows/build.yml
+    with:
+      os: ${{ matrix.os }}
+  test:
+    needs: build
+    uses: ./.github/workflows/test.yml
   release:
     name: Release
+    needs: [build, test]
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
@@ -13,18 +24,13 @@ jobs:
         uses: actions/setup-node@v2
         with:
           node-version: '14'
-      - name: Build
-        run: |
-          npm ci
-          npm run build:dist
-      - name: Test
-        run: |
-          npm run lint
-          npm run test
-      - name: Coverage
-        uses: coverallsapp/github-action@v1.1.2
+      - name: Install dependencies
+        run: npm ci
+      - name: Download artifacts
+        uses: actions/download-artifact@v3
         with:
-          github-token: ${{ secrets.GITHUB_TOKEN }}
+          name: dist-ubuntu-latest
+          path: dist
       - name: Release
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/test.yml

@@ -0,0 +1,34 @@
+name: Test
+on: workflow_call
+jobs:
+  test:
+    name: Test
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest]
+        timezone: [Etc/GMT, Asia/Shanghai, America/New_York]
+    runs-on: ${{ matrix.os }}
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+        with:
+          fetch-depth: 0
+      - name: Setup Node.js
+        uses: actions/setup-node@v2
+        with:
+          node-version: '14'
+      - name: Install dependencies
+        run: npm ci
+      - name: Download artifacts
+        uses: actions/download-artifact@v3
+        with:
+          name: dist-${{ matrix.os }}
+          path: dist
+      - name: Run Test
+        run: TZ=${{ matrix.timezone }} npm test
+      - name: Archive npm failure logs
+        uses: actions/upload-artifact@v2
+        if: failure()
+        with:
+          name: npm-logs
+          path: ~/.npm/_logs

.gitignore

@@ -9,6 +9,9 @@ coverage/
 
 # modules
 node_modules/
+
+# tmp
+docs/public/js/liquid.browser.min.js
 dist/
 demo/*/package-lock.json
 demo/*/yarn.json

CONTRIBUTING.md

@@ -16,14 +16,14 @@
   - Ensure the build runs because the Husky pre-commit hook checks it
     - `npm run check` checks runs the build, tests, lint and perf tests
     - `commitlint` checks the commit message format
-    
+
     If there is a problem you will see it in the pre-commit hook output.
     In VS Code, this output will be shown in a new file in a new tab if the
     pre-commit hook fails.
     If you want to check the commit message without using the VS Code Source
     Control UI, you can run `echo "feat: my commit message" > npx commitlint`
     directly.
-    
+
   - `git switch -c your_branch_name` (do this in your fork not the main repo)
   - `git add .`
   - `git commit -m "feat: Adding my change"`
@@ -33,32 +33,24 @@
 ## Playground
 
 The Playground runs off the `docs` directory.
-`npm run:docs` is used to build it and that's included in `npm run build`.
+`npm run build:docs` is used to build it.
 
-To start the site locally, go to `docs` and run `npm start`, then visit
+Then, to start the site locally, go to `docs` and run `npm start`, then visit
 http://localhost:4000/playground.html.
 
-At the moment, the Playground uses the latest NPM version of the LiquidJS
-library instead of using the built artifact produced by `npm run build:dist`
-(also included in `npm run build`).
+The Playground uses a local built LiquidJS, which is created during `npm run build:docs`.
+To update that, you'll need to run `./bin/build-docs-liquid.sh` each time after making changes.
+Then refresh the Playground site for the changes to take effect.
+
+## Performance
+
+If your change can have a performance impact, you can update and run performance cases under `benchmark/`.
 
-To use the Playground with the local build of the library, make these changes:
+1. `npm run build:cjs` to build a CommonJS bundle for the perf test.
+2. `npm run perf:diff` to check whether there's a regression compared against `liquidjs@latest`
 
-- Copy `dist/liquid.browser.min.js` to `docs/public/js/liquid.browser.min.js`
-- Open `docs/themes/navy/layout/partial/after_footer.swig`
-- Remove `https://cdn.jsdelivr.net/npm/liquidjs/dist/liquid.browser.min.js` line
-- Add `{{ js('liquid.browser.min.js') }}` line before `{{ js('js/main') }}` line
-- Refresh the Playground site for the changes to take effect
-- Replace `liquid.browser.min.js` each time after making changes and building
+Further more, `benchmark/` contains different cases to check its ops/sec. Useful when debugging perf regressions, to use it:
 
-```diff
- {% if page.layout === 'playground' %}
--<script src="https://cdn.jsdelivr.net/npm/liquidjs/dist/liquid.browser.min.js"></script>
- <script src="https://cdn.jsdelivr.net/npm/ace-builds@1.4.8/src-min/ace.js"></script>
- {% endif %}
-
-+{{ js('js/liquid.browser.min.js') }}
- {{ js('js/main') }}
-
- <script src="https://cdn.jsdelivr.net/npm/docsearch.js@2/dist/cdn/docsearch.min.js"></script>
-```
+- `cd benchmark` go into benchmark project
+- `npm ci` install dependencies
+- `npm start` run the cases

bin/build-docs-liquid.sh

@@ -0,0 +1,5 @@
+#!/usr/bin/env bash
+
+BUNDLES=min npm run build
+cp dist/liquid.browser.min.js docs/public/js/
+