Skip to content

Commit 75b4593

Browse files
committed
- Linting: Switch to Unix line breaks and other changes per ash-nazg, including linting Markdown JS
- Linting: Use recommended `.json` extension - Linting: Switch to ash-nazg - Linting: Add lgtm.yml file for lgtm.com - npm: Update devDeps, and update per security audit - npm: Bump to 0.19.0
1 parent 4de1d17 commit 75b4593

37 files changed

+4458
-5266
lines changed

Diff for: .eslintignore

+1
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
1+
node_modules
12
dist

Diff for: .eslintrc

-12
This file was deleted.

Diff for: .eslintrc.json

+64
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,64 @@
1+
{
2+
"extends": ["ash-nazg/sauron-node"],
3+
"overrides": [
4+
{
5+
"files": "src/jsonpath.js",
6+
"globals": {
7+
"require": "readonly",
8+
"module": "readonly"
9+
}
10+
},
11+
{
12+
"files": ["*.md"],
13+
"rules": {
14+
"import/unambiguous": 0,
15+
"import/no-commonjs": 0,
16+
"import/no-unresolved": ["error", {"ignore": ["jsonpath-plus"]}],
17+
"no-undef": 0,
18+
"no-unused-vars": ["error", {
19+
"varsIgnorePattern": "json|result"
20+
}],
21+
"node/no-missing-require": ["error", {
22+
"allowModules": ["jsonpath-plus"]
23+
}],
24+
"node/no-missing-import": ["error", {
25+
"allowModules": ["jsonpath-plus"]
26+
}]
27+
}
28+
},
29+
{
30+
"files": ["test/**"],
31+
"globals": {
32+
"require": "readonly",
33+
"module": "readonly"
34+
},
35+
"parserOptions": {
36+
"sourceType": "script"
37+
},
38+
"rules": {
39+
"strict": ["error", "global"],
40+
"import/no-commonjs": 0,
41+
"import/unambiguous": 0,
42+
"quotes": 0
43+
}
44+
},
45+
{
46+
"files": ["test-helpers/testLoading.js"],
47+
"env": {"node": false, "browser": true},
48+
"parserOptions": {
49+
"sourceType": "module"
50+
},
51+
"rules": {
52+
"no-shadow": ["error", {"allow": ["require", "module"]}]
53+
}
54+
}
55+
],
56+
"rules": {
57+
"indent": ["error", 4, {"outerIIFEBody": 0}],
58+
"promise/prefer-await-to-callbacks": 0,
59+
"quote-props": 0,
60+
"require-unicode-regexp": 0,
61+
"max-len": 0,
62+
"require-jsdoc": 0
63+
}
64+
}

Diff for: .npmignore

+1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
node_modules
22
test
33
test-helpers
4+
rollup.config.js

Diff for: CHANGES.md

+6-1
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,17 @@
11
# jsonpath-plus changes
22

3-
## ?
3+
## 0.19.0 (May 16, 2019)
44

55
- Docs (README): Indicate features, including performance (removing old note)
66
- Docs (README): Add headings for setup and fix headings levels
77
- Docs (README): Indicate parent selector was not present in original spec
88
(not just not documented)
99
- Docs (README): Fix escaping
10+
- Linting: Switch to Unix line breaks and other changes per ash-nazg, including linting Markdown JS
11+
- Linting: Use recommended `.json` extension
12+
- Linting: Switch to ash-nazg
13+
- Linting: Add lgtm.yml file for lgtm.com
14+
- npm: Update devDeps, and update per security audit
1015

1116
## 0.18.1 (May 14, 2019)
1217

Diff for: README.md

+5-3
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,8 @@ npm install jsonpath-plus
5151

5252
```js
5353
const {JSONPath} = require('jsonpath-plus');
54-
const result = JSONPath({path: '...', json: ...});
54+
55+
const result = JSONPath({path: '...', json});
5556
```
5657

5758
### Browser
@@ -83,14 +84,15 @@ Or if you are bundling your JavaScript (e.g., with Rollup), just use:
8384

8485
```js
8586
import JSONPath from 'jsonpath-plus';
86-
const result = JSONPath({path: '...', json: ...});
87+
88+
const result = JSONPath({path: '...', json});
8789
```
8890

8991
## Usage
9092

9193
The full signature available is:
9294

93-
```js
95+
```
9496
const result = JSONPath([options,] path, json, callback, otherTypeCallback);
9597
```
9698

0 commit comments

Comments
 (0)