merged scratch-gui: d845229

Author: takaokouji

.circleci/config.yml

@@ -1,4 +1,8 @@
 version: 2.1
+orbs:
+  browser-tools: circleci/browser-tools@1.2.4
+  commitlint: conventional-changelog/commitlint@1.0.0
+  node: circleci/node@5.0.2
 aliases:
   - &save_git_cache
     save_cache:
@@ -40,20 +44,64 @@ aliases:
         - v3-npm-
   - &defaults
     docker:
-      - image: circleci/node:10-browsers
+      # TODO: fix scratch-audio and change this to `cimg/node:lts-browsers`
+      - image: cimg/node:14.20-browsers
+        auth:
+          username: $DOCKERHUB_USERNAME
+          password: $DOCKERHUB_PASSWORD
+    executor: node/default
     working_directory: ~/repo
 
 jobs:
-  setup:
+  build-test-no-cache:
     <<: *defaults
+    environment:
+      DETECT_CHROMEDRIVER_VERSION: "true"
+      JEST_JUNIT_OUTPUT_DIR: test-results
+      NODE_OPTIONS: --max-old-space-size=4000
     steps:
       - *restore_git_cache
       - checkout
-      - *restore_npm_cache
+      - browser-tools/install-chrome # must be before node/install-packages
+      - node/install-packages
       - run: |
           rm -rf node_modules/scratch-vm
           npm install
           (cd node_modules/scratch-vm && npm install && $(npm bin)/webpack --colors --bail --silent)
+      - run:
+          name: Lint
+          command: npm run test:lint -- --quiet --output-file test-results/eslint-results.xml --format junit
+      - run:
+          name: Unit
+          environment:
+            JEST_JUNIT_OUTPUT_NAME: unit-results.xml
+          command: npm run test:unit -- --reporters="default" --reporters="jest-junit" --coverage --coverageReporters=text --coverageReporters=lcov --maxWorkers="2"
+      - run:
+          name: Build
+          environment:
+            NODE_ENV: production
+          command: npm run build
+      - run:
+          name: Integration
+          environment:
+              JEST_JUNIT_OUTPUT_NAME: integration-results.xml
+          command: |
+              google-chrome --version
+              npx --no -- chromedriver --version
+              npm run test:integration -- --reporters="default" --reporters="jest-junit"
+      - store_artifacts:
+          path: coverage
+      - store_test_results:
+          path: test-results
+  setup:
+    <<: *defaults
+    environment:
+      DETECT_CHROMEDRIVER_VERSION: "true"
+    steps:
+      - *restore_git_cache
+      - checkout
+      - browser-tools/install-chrome # must be before node/install-packages
+      - node/install-packages
       - *save_git_cache
       - *save_npm_cache
   lint:
@@ -120,16 +168,35 @@ jobs:
       - checkout
       - *restore_npm_cache
       - *restore_build_cache
+      - browser-tools/install-chrome
       - run:
           name: Integration
           environment:
               JEST_JUNIT_OUTPUT_DIR: test-results/integration
           command: |
+            google-chrome --version
+            npx --no -- chromedriver --version
             export TESTFILES=$(circleci tests glob "test/integration/*.test.js" | circleci tests split --split-by=timings)
             $(npm bin)/jest ${TESTFILES} --reporters="default" --reporters="jest-junit" --runInBand
       - store_test_results:
           path: test-results
 
+  deploy-npm:
+    <<: *defaults
+    steps:
+      - *restore_git_cache
+      - checkout
+      - *restore_npm_cache
+      - *restore_dist_cache
+      - run:
+          name: adjust config for hotfix if necessary
+          command: |
+            # double brackets are important for matching the wildcard
+            if [[ "$CIRCLE_BRANCH" == hotfix/* ]]; then
+              sed -e "s|hotfix/REPLACE|${CIRCLE_BRANCH}|" --in-place release.config.js
+            fi
+      - run: npx semantic-release
+
   deploy-gh-pages:
     <<: *defaults
     steps:
@@ -145,28 +212,95 @@ jobs:
 
 workflows:
   version: 2
+  commitlint:
+    jobs:
+      - commitlint/lint:
+          target-branch: develop
+  push-translations:
+    triggers:
+      - schedule:
+          cron: 0 0 * * * # daily at 12 UTC, 8 ET
+          filters:
+            branches:
+              only:
+                - develop
+    jobs:
+      - setup:
+          context:
+            - dockerhub-credentials
+      - push-translations:
+          context:
+            - dockerhub-credentials
+          requires:
+              - setup
+
+  build-test-no-deploy:
+    jobs:
+      - build-test-no-cache:
+          context:
+            - dockerhub-credentials
+          filters:
+            branches:
+              ignore:
+                - master
+                - develop
+                - /^hotfix\/.*/
   build-test-deploy:
     jobs:
-      - setup
+      - setup:
+          context:
+            - dockerhub-credentials
+          filters:
+            branches:
+              only:
+                - master
+                - develop
+                - /^hotfix\/.*/
       - lint:
+          context:
+            - dockerhub-credentials
           requires:
             - setup
       - unit:
+          context:
+            - dockerhub-credentials
           requires:
             - setup
       - build:
+          context:
+            - dockerhub-credentials
           requires:
             - setup
       - integration:
+          context:
+            - dockerhub-credentials
           requires:
             - build
       - store_build:
+          context:
+            - dockerhub-credentials
           requires:
             - build
+          filters:
+            branches:
+              only:
+                - master
+                - develop
+                - /^hotfix\/.*/
       - store_dist:
+          context:
+            - dockerhub-credentials
           requires:
             - build
-      - deploy-gh-pages:
+          filters:
+            branches:
+              only:
+                - master
+                - develop
+                - /^hotfix\/.*/
+      - deploy-npm:
+          context:
+            - dockerhub-credentials
           requires:
             - lint
             - unit
@@ -175,4 +309,20 @@ workflows:
           filters:
             branches:
               only:
+                - master
                 - develop
+                - /^hotfix\/.*/
+      - deploy-gh-pages:
+          context:
+            - dockerhub-credentials
+          requires:
+            - lint
+            - unit
+            - integration
+            - build
+          filters:
+            branches:
+              ignore:
+                - /^dependabot/.*/
+                - /^renovate/.*/
+                - /^pull/.*/ # don't deploy to  gh pages on PRs.

.gitattributes

@@ -20,6 +20,7 @@
 *.js text eol=lf
 *.js.map text eol=lf
 *.json text eol=lf
+*.json5 text eol=lf
 *.jsx text eol=lf
 *.md text eol=lf
 *.vert text eol=lf

.husky/.gitattributes

@@ -0,0 +1 @@
+* text eol=lf

.husky/commit-msg

@@ -0,0 +1,4 @@
+#!/usr/bin/env sh
+. "$(dirname -- "$0")/_/husky.sh"
+
+npx --no-install commitlint --edit "$1"