Merge a0c7e87 into 1417c1a

.eslintrc.json

@@ -1,6 +1,6 @@
 {
    "extends": ["semistandard"],
-   "ignorePatterns": ["dist", "docs"],
+   "ignorePatterns": ["dist"],
    "rules": {
       "arrow-spacing": "error",
       "quotes": ["error", "single"],

.github/testcafe.json

@@ -1,5 +1,6 @@
 {
   "src": ["tests/acceptance/acceptancesuites/*.js", "!tests/acceptance/acceptancesuites/searchbaronlysuite.js"],
   "appCommand": "npx serve -l tcp://0.0.0.0:9999",
-  "appInitDelay": 4000
+  "appInitDelay": 4000,
+  "skipJsErrors": true
 }

.github/workflows/acceptance.yml

@@ -14,10 +14,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Use Node.js 14
+      - name: Use Node.js 16
         uses: actions/setup-node@v2
         with:
-          node-version: 14
+          node-version: 16
           cache: 'npm'
       - run: npm ci
       - name: Download build-output artifact
@@ -32,10 +32,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Use Node.js 14
+      - name: Use Node.js 16
         uses: actions/setup-node@v2
         with:
-          node-version: 14
+          node-version: 16
           cache: 'npm'
       - run: npm ci
       - name: Download build-output artifact

.github/workflows/acceptance_search_bar.yml

@@ -14,10 +14,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Use Node.js 14
+      - name: Use Node.js 16
         uses: actions/setup-node@v2
         with:
-          node-version: 14
+          node-version: 16
           cache: 'npm'
       - run: npm ci
       - name: Download build-output artifact

.github/workflows/build.yml

@@ -14,10 +14,12 @@ jobs:
 
     steps:
     - uses: actions/checkout@v2
-    - name: Use Node.js 14
+      with:
+        fetch-depth: 0
+    - name: Use Node.js 16
       uses: actions/setup-node@v2
       with:
-        node-version: 14
+        node-version: 16
         cache: 'npm'
     - run: npm ci
     - run: npm run ${{ inputs.build_script }}

.github/workflows/build_and_deploy.yml

@@ -5,7 +5,6 @@ on:
     branches-ignore:
       - develop
       - master
-      - support/**
       - hotfix/**
       - feature/**-i18n
       - release/**

.github/workflows/build_and_deploy_i18n.yml

@@ -4,7 +4,6 @@ on:
   push:
     branches:
       - develop
-      - support/**
       - hotfix/**
       - feature/**-i18n
       - release/**

.github/workflows/coverage.yml

@@ -9,23 +9,9 @@ on:
   pull_request:
 
 jobs:
-  Coveralls:
-    runs-on: ubuntu-latest
-
-    strategy:
-      matrix:
-        node-version: [14.x]
-
-    steps:
-    - uses: actions/checkout@v2
-    - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v2
-      with:
-        node-version: ${{ matrix.node-version }}
-    - run: npm ci
-    - run: npx gulp templates
-    - run: npx jest --coverage
-    - name: Coveralls
-      uses: coverallsapp/github-action@master
-      with:
-        github-token: ${{ secrets.GITHUB_TOKEN }}
+  call_coveralls:
+    uses: yext/slapshot-reusable-workflows/.github/workflows/coverage.yml@v1
+    with:
+      test_script: npx gulp templates && npx jest --coverage
+    secrets:
+      caller_github_token: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/miscellaneous_tests.yml

@@ -9,10 +9,10 @@ jobs:
 
     steps:
       - uses: actions/checkout@v2
-      - name: Use Node.js 14
+      - name: Use Node.js 16
         uses: actions/setup-node@v2
         with:
-          node-version: 14
+          node-version: 16
           cache: 'npm'
       - run: npm ci
       - run: sudo apt-get install -qq gettext

.github/workflows/percy_snapshots.yml

@@ -6,23 +6,10 @@ on:
   pull_request:
 
 jobs:
-  build:
-
-    runs-on: ubuntu-latest
-
-    strategy:
-      matrix:
-        node-version: [15.x]
-
-    steps:
-    - uses: actions/checkout@v2
-    - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v2
-      with:
-        node-version: ${{ matrix.node-version }}
-    - run: npm ci
-    - run: npm run build
-    - name: Percy Snapshots
-      run: npx percy exec -- node tests/acceptance/percy/snapshots.js
-      env:
-        PERCY_TOKEN: ${{ secrets.PERCY_TOKEN }}
+  call_percy_snapshots:
+    uses: yext/slapshot-reusable-workflows/.github/workflows/percy_snapshots.yml@v1
+    with:
+      snapshots_script_path: tests/acceptance/percy/snapshots.js
+      fetch_depth: 0
+    secrets:
+      PERCY_TOKEN: ${{ secrets.PERCY_TOKEN }}

.github/workflows/sync_develop_and_main.yml

@@ -1,27 +1,11 @@
-name: create PR from main to develop
+name: Create PR from main to develop
 
 on:
   push:
     branches: [main, master]
 
-permissions:
-  contents: read
-  pull-requests: write
-
 jobs:
-  createPullRequest:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v2
-      - name: extract package version
-        id: vars
-        run: |
-          PACKAGE_VERSION="v$(cat ./package.json | grep version | head -1 | awk -F: '{ print $2 }' | sed 's/[",]//g' | tr -d '[[:space:]]')"
-          echo ::set-output name=tag::${PACKAGE_VERSION}
-      - uses: repo-sync/pull-request@v2
-        with:
-          source_branch: "${{ github.event.repository.default_branch }}"
-          destination_branch: "develop"
-          pr_title: "Merge ${{ github.event.repository.default_branch }} (${{ steps.vars.outputs.tag }}) into develop"
-          pr_body: "Merge ${{ github.event.repository.default_branch }} (${{ steps.vars.outputs.tag }}) into develop"
-          github_token: ${{ secrets.GITHUB_TOKEN }}
+  call_sync_develop_and_main:
+    uses: yext/slapshot-reusable-workflows/.github/workflows/sync_develop_and_main.yml@v1
+    secrets:
+      caller_github_token: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/third_party_notices_check.yml

@@ -3,20 +3,7 @@ name: Check Third Party Notices File
 on: pull_request
 
 jobs:
-  license-check:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/setup-node@v2
-        with:
-          node-version: 14.x
-      - run: npm install -g generate-license-file
-      - uses: actions/checkout@v2
-      - run: npm ci
-      - run: generate-license-file --input package.json --output ./THIRD-PARTY-NOTICES
-      - name: Update THIRD-PARTY-NOTICES
-        uses: EndBug/add-and-commit@v7
-        with:
-          message: "Automated update to THIRD-PARTY-NOTICES from github action's 3rd party notices check"
-          add: 'THIRD-PARTY-NOTICES'
-          push: true
-          default_author: github_actions
+  call_notices_check:
+    uses: yext/slapshot-reusable-workflows/.github/workflows/third_party_notices_check.yml@v1
+    secrets:
+      REPO_SCOPED_TOKEN: ${{ secrets.BOT_REPO_SCOPED_TOKEN }}

.github/workflows/unit_test.yml

@@ -9,10 +9,10 @@ jobs:
 
     steps:
       - uses: actions/checkout@v2
-      - name: Use Node.js 14
+      - name: Use Node.js 16
         uses: actions/setup-node@v2
         with:
-          node-version: 14
+          node-version: 16
           cache: 'npm'
       - run: npm ci
       - name: Download build-output artifact

.github/workflows/version-update.yml

@@ -0,0 +1,11 @@
+name: Update package version for release & hotfix branches
+
+on:
+  push:
+    branches: [release/*, hotfix/*]
+
+jobs:
+  call_version_update:
+    uses: yext/slapshot-reusable-workflows/.github/workflows/version_update.yml@v1
+    secrets:
+      caller_github_token: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/wcag_test.yml

@@ -5,18 +5,7 @@ on:
     branches: [develop, hotfix/*, release/*, support/*]
 
 jobs:
-  WCAG-test:
-    runs-on: ubuntu-latest
-
-    strategy:
-      matrix:
-        node-version: [14.x]
-
-    steps:
-    - uses: actions/checkout@v2
-    - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v2
-      with:
-        node-version: ${{ matrix.node-version }}
-    - run: npm ci
-    - run: npm run wcag
+  call_wcag_test:
+    uses: yext/slapshot-reusable-workflows/.github/workflows/wcag_test.yml@v1
+    with:
+      fetch_depth: 0

.postcssrc.json

@@ -6,7 +6,7 @@
       "mediaQuery": true
     },
     "autoprefixer": {
-      "browsers": ["last 2 versions"],
+      "overrideBrowserslist": ["last 2 versions"],
       "grid": true
     },
     "cssnano": true

.size-limit.js

@@ -1,11 +1,11 @@
 module.exports = [
   {
     path: 'dist/answers.min.js',
-    limit: '180 KB'
+    limit: '190 KB'
   },
   {
     path: 'dist/answers-modern.min.js',
-    limit: '150kb'
+    limit: '155 KB'
   },
   {
     path: 'dist/answerstemplates.compiled.min.js',

.stylelintrc.json

@@ -1,15 +1,45 @@
 {
-    "plugins": [
-        "stylelint-scss"
-    ],
+    "extends": "stylelint-config-standard-scss",
     "rules": {
         "color-no-invalid-hex": true,
-        "function-calc-no-invalid": true,
         "unit-no-unknown": true,
         "property-no-unknown": true,
         "declaration-block-no-duplicate-properties": true,
         "no-duplicate-selectors": true,
+        "alpha-value-notation": null,
+        "block-closing-brace-newline-before": null,
+        "block-closing-brace-newline-after": null,
+        "value-keyword-case": null,
+        "max-line-length": null,
+        "string-quotes": "double",
+        "rule-empty-line-before": null,
+        "length-zero-no-unit": null,
+        "selector-descendant-combinator-no-non-space": null,
+        "color-hex-case": "upper",
+        "declaration-empty-line-before": null,
+        "no-eol-whitespace": [true, { "ignore": ["empty-lines"] }],
+        "color-function-notation": "legacy",
+        "no-descending-specificity": null,
+        "selector-type-no-unknown": null,
+        "keyframes-name-pattern": null,
+        "comment-whitespace-inside": null,
+        "selector-no-vendor-prefix": null,
+        "declaration-block-no-shorthand-property-overrides": null,
+        "comment-empty-line-before": null,
+        "selector-list-comma-newline-after": null,
+        "function-calc-no-unspaced-operator": null,
+        "custom-property-empty-line-before": null,
+        "declaration-block-no-redundant-longhand-properties": null,
+        "custom-property-pattern": null,
+        "scss/at-mixin-pattern": null,
+        "scss/dollar-variable-pattern": null,
+        "scss/operator-no-unspaced": null,
         "scss/dollar-variable-default": [true, { "ignore": "local" }],
+        "scss/dollar-variable-empty-line-before": null,
+        "block-opening-brace-space-before": null,
+        "selector-class-pattern": null,
+        "at-rule-empty-line-before": null,
+        "scss/at-mixin-argumentless-call-parentheses": "always",
         "scss/at-import-no-partial-leading-underscore": true,
         "scss/at-import-partial-extension": "never"
     }