ci: add checking solidity linting to ci

.github/workflows/ci.yaml

@@ -6,6 +6,26 @@ on:
     branches: [main]
   workflow_dispatch:
 jobs:
+  lint:
+    name: Lint
+    runs-on: ubuntu-18.04
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v2
+
+      - name: Setup Node.js environment
+        uses: actions/setup-node@v2
+        with:
+          node-version: 16
+          cache: "yarn"
+
+      - name: Install dependencies
+        # Hack to get around failing "ethereumjs-abi The remote archive doesn't match the expected checksum" error
+        run: YARN_CHECKSUM_BEHAVIOR=update yarn
+
+      - name: Lint solidity
+        run: yarn lint
+
   build:
     runs-on: ubuntu-18.04
     env:

package.json

@@ -16,7 +16,8 @@
     "hardhat:test": "hardhat test",
     "hardhat:compile": "hardhat compile",
     "hardhat:coverage": "hardhat coverage",
-    "lint:sol": "solhint --max-warnings 0 \"contracts/**/*.sol\"",
+    "lint": "yarn lint:sol",
+    "lint:sol": "solhint \"contracts/**/*.sol\"",
     "lint:sol:fix": "prettier --write \"contracts/**/*.sol\"",
     "build": "rm -rf dist && tsc --declaration && hardhat compile && cp -r ./{package.json,yarn.lock,artifacts,networks,contracts,deployments,copy_contracts.sh} dist/",
     "publish:dist": "yarn build && yarn publish dist -f --access public",