Merge branch 'main' into remove_docker_build

# Conflicts:
#	action/index.js
#	src/model/docker.js

.github/workflows/main.yml

@@ -12,14 +12,14 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - uses: actions/setup-node@v1.1.2
+      - uses: actions/setup-node@v2
         with:
           node-version: 12.x
       - run: yarn
       - run: yarn lint
       - run: yarn test
       - run: yarn build || { echo "build command should always succeed" ; exit 61; }
-      - run: yarn build --quiet && git diff --quiet action || { echo "action should be auto generated" ; exit 62; }
+      - run: yarn build --quiet && git diff --quiet action || { echo "ERROR - index.js is different from repository version. Forgot to run `yarn build`?" ; exit 62; }
 
   testAllModesLikeInTheReadme:
     name: Test in ${{ matrix.testMode }} on version ${{ matrix.unityVersion }}
@@ -38,7 +38,7 @@ jobs:
       - uses: actions/checkout@v2
         with:
           lfs: true
-      - uses: actions/cache@v1.1.0
+      - uses: actions/cache@v2
         with:
           path: ${{ matrix.projectPath }}/Library
           key: Library-${{ matrix.projectPath }}

.prettierrc.json

@@ -2,5 +2,6 @@
   "semi": true,
   "singleQuote": true,
   "trailingComma": "all",
-  "printWidth": 100
+  "printWidth": 100,
+  "arrowParens": "avoid"
 }

CONTRIBUTING.md

@@ -31,8 +31,8 @@ You have [Node](https://nodejs.org/) installed at v12.2.0+ and [Yarn](https://ya
 Please note that commit hooks will run automatically to perform some tasks;
 
 - format your code
-- run tests
-- build distributable files
+- run tests & lint - `yarn lint && yarn test`
+- build distributable files - `yarn build`
 
 #### License
 

package.json

@@ -13,30 +13,30 @@
     "test": "jest"
   },
   "dependencies": {
-    "@actions/core": "^1.2.1",
-    "@actions/exec": "1.0.3",
-    "@actions/github": "^2.1.1"
+    "@actions/core": "^1.2.6",
+    "@actions/exec": "1.0.4",
+    "@actions/github": "^4.0.0"
   },
   "devDependencies": {
-    "@babel/cli": "7.8.4",
-    "@babel/core": "7.8.7",
-    "@babel/preset-env": "7.8.7",
-    "@zeit/ncc": "0.21.1",
+    "@babel/cli": "7.12.10",
+    "@babel/core": "7.12.10",
+    "@babel/preset-env": "7.12.11",
+    "@zeit/ncc": "0.22.3",
     "babel-eslint": "10.1.0",
-    "eslint": "6.8.0",
-    "eslint-config-airbnb": "18.0.1",
-    "eslint-config-prettier": "6.10.0",
-    "eslint-plugin-flowtype": "4.6.0",
-    "eslint-plugin-import": "2.20.1",
-    "eslint-plugin-jsx-a11y": "6.2.3",
-    "eslint-plugin-prettier": "3.1.2",
-    "eslint-plugin-react": "7.19.0",
-    "eslint-plugin-unicorn": "16.1.1",
-    "husky": "4.2.3",
-    "jest": "25.1.0",
-    "lint-staged": "10.0.8",
-    "lodash-es": "4.17.15",
-    "prettier": "1.19.1"
+    "eslint": "7.17.0",
+    "eslint-config-airbnb": "18.2.1",
+    "eslint-config-prettier": "7.1.0",
+    "eslint-plugin-flowtype": "5.2.0",
+    "eslint-plugin-import": "2.22.1",
+    "eslint-plugin-jsx-a11y": "6.4.1",
+    "eslint-plugin-prettier": "3.3.1",
+    "eslint-plugin-react": "7.22.0",
+    "eslint-plugin-unicorn": "25.0.1",
+    "husky": "4.3.7",
+    "jest": "26.6.3",
+    "lint-staged": "10.5.3",
+    "lodash-es": "4.17.20",
+    "prettier": "2.2.1"
   },
   "husky": {
     "hooks": {

src/model/docker.js

@@ -51,7 +51,7 @@ class Docker {
         ${useHostNetwork ? '--net=host' : ''} \
         ${image} /bin/bash -c "chmod -R +x /github/action && /github/action/entrypoint.sh"`;
 
-    await exec(command, null, { silent });
+    await exec(command, undefined, { silent });
   }
 }
 

src/model/image-tag.test.js

@@ -23,7 +23,7 @@ describe('ImageTag', () => {
       expect(() => new ImageTag({ version })).not.toThrow();
     });
 
-    test.each(['some version', '', 1, null])('throws for incorrect versions %p', version => {
+    test.each(['some version', '', 1, undefined])('throws for incorrect versions %p', version => {
       expect(() => new ImageTag({ version })).toThrow();
     });
   });

src/model/input.js

@@ -8,7 +8,7 @@ class Input {
   }
 
   static isValidFolderName(folderName) {
-    const validFolderName = new RegExp(/^(\.|\.\/)?(\.?\w+([-_]?\w+)*\/?)*$/);
+    const validFolderName = new RegExp(/^(\.|\.\/)?(\.?\w+([_-]?\w+)*\/?)*$/);
 
     return validFolderName.test(folderName);
   }