[test] Add jsonlint to CI (#13126)

* [test] Add jsonlint to CI * Update package.json
mui · Oct 6, 2018 · a387b49 · a387b49
1 parent 1f01a54
commit a387b49
Show file tree

Hide file tree

Showing 5 changed files with 22 additions and 4 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -84,6 +84,9 @@ jobs:
       - run:
           name: Lint
           command: yarn lint
+      - run:
+          name: Lint JSON
+          command: yarn jsonlint
       - run:
           name: Tests TypeScript definitions
           command: yarn typescript

diff --git a/package.json b/package.json
@@ -32,6 +32,8 @@
     "docs:size-why": "DOCS_STATS_ENABLED=true yarn docs:build",
     "docs:build-sw": "babel-node ./docs/scripts/buildServiceWorker.js",
     "docs:deploy": "git push material-ui-docs master:latest",
+    "jsonlint": "yarn --silent jsonlint:files | xargs -n1 jsonlint -q -c && echo \"jsonlint: no lint errors\"",
+    "jsonlint:files": "find . -name \"*.json\" | grep -v -f .eslintignore",
     "lint": "eslint . --cache && echo \"eslint: no lint errors\"",
     "lint:fix": "eslint . --cache --fix && echo \"eslint: no lint errors\"",
     "prettier": "yarn --silent prettier:files | xargs prettier --write",
@@ -108,6 +110,7 @@
     "gm": "^1.23.0",
     "isomorphic-fetch": "^2.2.1",
     "jsdom": "^12.0.0",
+    "jsonlint": "^1.6.3",
     "jss-rtl": "^0.2.1",
     "karma": "^3.0.0",
     "karma-browserstack-launcher": "^1.3.0",

diff --git a/tsconfig.json b/tsconfig.json
@@ -16,5 +16,5 @@
       "@material-ui/lab": ["./material-ui-lab/src"],
       "@material-ui/lab/*": ["./material-ui-lab/src/*"]
     }
-  },
+  }
 }
diff --git a/tslint.json b/tslint.json
@@ -4,8 +4,7 @@
     "jsRules": {},
     "rules": {
         "file-name-casing": false,
-        // This can be tricky in most cases and doesn't hurt if it is done one to many
         "no-unnecessary-generics": false,
         "semicolon": [true, "always", "ignore-bound-class-methods"]
     }
-}
+}
diff --git a/yarn.lock b/yarn.lock
@@ -1249,6 +1249,11 @@
   resolved "https://registry.yarnpkg.com/@xtuc/long/-/long-4.2.1.tgz#5c85d662f76fa1d34575766c5dcd6615abcd30d8"
   integrity sha512-FZdkNBDqBRHKQ2MEbSC17xnPFOhZxeJ2YGSfr2BKf3sujG49Qe3bB+rGCwQfIaA7WHnGeGkSijX4FuBCdrzW/g==
 
+JSV@^4.0.x:
+  version "4.0.2"
+  resolved "https://registry.yarnpkg.com/JSV/-/JSV-4.0.2.tgz#d077f6825571f82132f9dffaed587b4029feff57"
+  integrity sha1-0Hf2glVx+CEy+d/67Vh7QCn+/1c=
+
 abab@^2.0.0:
   version "2.0.0"
   resolved "https://registry.yarnpkg.com/abab/-/abab-2.0.0.tgz#aba0ab4c5eee2d4c79d3487d85450fb2376ebb0f"
@@ -6870,6 +6875,14 @@ jsonify@~0.0.0:
   resolved "https://registry.yarnpkg.com/jsonify/-/jsonify-0.0.0.tgz#2c74b6ee41d93ca51b7b5aaee8f503631d252a73"
   integrity sha1-LHS27kHZPKUbe1qu6PUDYx0lKnM=
 
+jsonlint@^1.6.3:
+  version "1.6.3"
+  resolved "https://registry.yarnpkg.com/jsonlint/-/jsonlint-1.6.3.tgz#cb5e31efc0b78291d0d862fbef05900adf212988"
+  integrity sha512-jMVTMzP+7gU/IyC6hvKyWpUU8tmTkK5b3BPNuMI9U8Sit+YAWLlZwB6Y6YrdCxfg2kNz05p3XY3Bmm4m26Nv3A==
+  dependencies:
+    JSV "^4.0.x"
+    nomnom "^1.5.x"
+
 jsprim@^1.2.2:
   version "1.4.1"
   resolved "https://registry.yarnpkg.com/jsprim/-/jsprim-1.4.1.tgz#313e66bc1e5cc06e438bc1b7499c2e5c56acb6a2"
@@ -8019,7 +8032,7 @@ node-releases@^1.0.0-alpha.12:
   dependencies:
     semver "^5.3.0"
 
-nomnom@^1.8.1:
+nomnom@^1.5.x, nomnom@^1.8.1:
   version "1.8.1"
   resolved "https://registry.yarnpkg.com/nomnom/-/nomnom-1.8.1.tgz#2151f722472ba79e50a76fc125bb8c8f2e4dc2a7"
   integrity sha1-IVH3Ikcrp55Qp2/BJbuMjy5Nwqc=
-Original file line number
+Diff line change
@@ Expand Up / @@ -16,5 +16,5 @@ @@
           "@material-ui/lab": ["./material-ui-lab/src"],
           "@material-ui/lab/*": ["./material-ui-lab/src/*"]
         }
-      },
+      }
     }