Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: package.json

Issue 29987559: Noissue - Add "ajv" peer-dependency (Closed)
Left Patch Set: Created Jan. 22, 2019, 1 p.m.
Right Patch Set: Created Jan. 22, 2019, 1:02 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « no previous file | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 { 1 {
2 "name": "buildtools", 2 "name": "buildtools",
3 "repository": "https://hg.adblockplus.org/buildtools", 3 "repository": "https://hg.adblockplus.org/buildtools",
4 "license": "MPL-2.0", 4 "license": "MPL-2.0",
5 "dependencies": { 5 "dependencies": {
6 "//": [ 6 "//": [
7 "'acorn' and 'ajv' are peer dependencies of manifoldjs, we don't use it", 7 "'acorn' and 'ajv' are peer dependencies of manifoldjs, we don't use",
kzar 2019/01/22 16:04:47 Aren't these called "transitive dependencies"?
kzar 2019/01/22 16:04:47 Did you mean to append the comma? I figure you mig
tlucas 2019/01/22 16:08:00 I did, i wanted to stick to 80 characters. for th
kzar 2019/01/22 16:12:36 Ah, gotya. Good to know, thanks.
8 " otherwise." 8 " them otherwise."
9 ], 9 ],
10 "acorn": "^6.0.0", 10 "acorn": "^6.0.0",
11 "ajv": "^5.0.0", 11 "ajv": "^5.0.0",
kzar 2019/01/22 16:04:47 (I have not looked into this ajv dependency, or ch
tlucas 2019/01/22 16:08:00 Quoting cron: "npm WARN ajv-keywords@2.1.1 require
12 "jsdoc": "3.5.5", 12 "jsdoc": "3.5.5",
13 "manifoldjs": "0.7.6", 13 "manifoldjs": "0.7.6",
14 "memory-fs": "0.4.1", 14 "memory-fs": "0.4.1",
15 "webpack": "3.6.0" 15 "webpack": "3.6.0"
16 }, 16 },
17 "scripts": { 17 "scripts": {
18 "jsdoc": "jsdoc", 18 "jsdoc": "jsdoc",
19 "build-edge": "manifoldjs -l error -p edgeextension -f edgeextension -d $EXT _FOLDER -m $SRC_FOLDER/manifest.json", 19 "build-edge": "manifoldjs -l error -p edgeextension -f edgeextension -d $EXT _FOLDER -m $SRC_FOLDER/manifest.json",
20 "package-edge": "manifoldjs -l error -p edgeextension package $EXT_FOLDER/*/ edgeextension/manifest/" 20 "package-edge": "manifoldjs -l error -p edgeextension package $EXT_FOLDER/*/ edgeextension/manifest/"
21 } 21 }
22 } 22 }
LEFTRIGHT
« no previous file | no next file » | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
This is Rietveld