Skip to content
This repository has been archived by the owner on Feb 4, 2021. It is now read-only.

Commit

Permalink
Browse files Browse the repository at this point in the history
Merge pull request #37 from mdboom/eslint
Lint the Javascript
  • Loading branch information
mdboom committed Mar 8, 2018
2 parents bb35250 + ecfaa26 commit 34ab4bf
Show file tree
Hide file tree
Showing 6 changed files with 1,212 additions and 36 deletions.
21 changes: 21 additions & 0 deletions .eslintrc.json
@@ -0,0 +1,21 @@
{
"env": {
"browser": true,
"jquery": true,
"amd": true
},
"plugins": [
"amd-imports"
],
"extends": "eslint:recommended",
"rules": {
"indent": [
"error",
4
],
"linebreak-style": [
"error",
"unix"
]
}
}
2 changes: 2 additions & 0 deletions .gitignore
Expand Up @@ -60,3 +60,5 @@ target/

#Ipython Notebook
.ipynb_checkpoints

node_modules/
14 changes: 14 additions & 0 deletions .travis.yml
Expand Up @@ -27,3 +27,17 @@ jobs:
on:
tags: true
repo: mozilla/jupyter-spark
matrix:
include:
- env: eslint
sudo: false
cache:
directories:
- node_modules # NPM packages
install:
- nvm install 8
- npm install
script:
- node ./node_modules/eslint/bin/eslint.js --ext .js src/jupyter_spark/static
allow_failures:
- env: eslint

0 comments on commit 34ab4bf

Please sign in to comment.