Merge pull request #564 from AugurProject/linting

Add linting to CI explicitly and fix current lint problem
AugurProject · Feb 27, 2018 · c5597b8 · c5597b8
2 parents 3dc92ef + 91eed23
commit c5597b8
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -17,7 +17,7 @@
     "test": "test"
   },
   "scripts": {
-    "lint": "npx solium --dir . --reporter=gcc",
+    "lint": "npx solium --dir source --reporter=gcc",
     "build:source": "npx tsc",
     "build:contracts": "node output/deployment/compileContracts.js",
     "build:interfaces": "node output/tools/generateContractInterfaces.js",

diff --git a/source/contracts/reporting/Universe.sol b/source/contracts/reporting/Universe.sol
@@ -376,7 +376,7 @@ contract Universe is DelegationTarget, Extractable, ITyped, Initializable, IUniv
                 .mul(_previousValue
                     .mul(_badMarkets)
                     .div(_totalMarkets)
-                    .sub(_previousValue / _targetDivisor));
+                .sub(_previousValue / _targetDivisor));
             _newValue = _newValue / (_targetDivisor - 1);
             _newValue = _newValue.add(_previousValue);
         }

diff --git a/source/support/Dockerfile b/source/support/Dockerfile
@@ -22,6 +22,7 @@ RUN pip install --upgrade pip setuptools \
 
 # Install basics of the application
 COPY .soliumrc.json /app/.soliumrc.json
+COPY .soliumignore /app/.soliumignore
 COPY tsconfig.json /app/tsconfig.json
 COPY typings/ /app/typings/
 COPY source/contracts/ /app/source/contracts/
@@ -34,6 +35,9 @@ COPY package.json /app/package.json
 
 RUN npm install
 
+# Lint
+RUN npm run lint
+
 # Build first chunk
 RUN npm run build:source
 RUN npm run build:contracts