Skip to content
This repository has been archived by the owner on Mar 7, 2023. It is now read-only.

Commit

Permalink
Add linting to CI
Browse files Browse the repository at this point in the history
  • Loading branch information
MadLittleMods committed May 9, 2019
1 parent 46eaecc commit b6f7406
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 1 deletion.
2 changes: 2 additions & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,8 @@ install:
- npm install --build-from-source

script:
- npm run lint

# Tests are disabled until we have tests which are suitable for CI and don't require manual interaction.
# - npm test
- echo "Skipping tests on CI, as they currently require manual interaction."
Expand Down
3 changes: 2 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@
"scripts": {
"install": "prebuild-install || node-gyp rebuild",
"prepublishOnly": "npm run validate",
"validate": "eslint **/*.js && npm test",
"lint": "eslint **/*.js",
"validate": "npm run lint && npm test",
"test": "jasmine ./test/test.js",
"prebuild": "prebuild --all --strip --verbose",
"rebuild": "node-gyp rebuild"
Expand Down
3 changes: 3 additions & 0 deletions test/.eslintrc
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@
"es6": true,
"jasmine": true
},
"parserOptions": {
"ecmaVersion": 2018
},
"rules": {
"no-console": 0,
}
Expand Down

0 comments on commit b6f7406

Please sign in to comment.