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

Delta Between Two Patch Sets: package.json

Issue 29808587: Issue 6717 - Part 1: run eslint through npm (Closed) Base URL: https://hg.adblockplus.org/adblockpluschrome/
Left Patch Set: Forgot the hgignore and gitignore Created June 15, 2018, 8:17 p.m.
Right Patch Set: Require eslint-config-eyeo 2.1.0 Created Aug. 14, 2018, 11:36 a.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 | « ext/background.js ('k') | 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": "adblockpluschrome", 2 "name": "adblockpluschrome",
3 "repository": "https://hg.adblockplus.org/adblockpluschrome", 3 "repository": "https://hg.adblockplus.org/adblockpluschrome",
4 "license": "GPL-3.0", 4 "license": "GPL-3.0",
5 "dependencies": {}, 5 "dependencies": {},
6 "devDependencies": { 6 "devDependencies": {
7 "eslint": "^3.19.0", 7 "eslint": "^5.1.0",
8 "eslint-config-eyeo": "1.0.4" 8 "eslint-config-eyeo": "^2.1.0"
9 }, 9 },
10 "scripts": { 10 "scripts": {
11 "eslint": "eslint *.js lib/ qunit/ ext/ chrome/", 11 "lint": "eslint *.js lib/ qunit/ ext/",
Sebastian Noack 2018/06/15 21:34:26 For reference, in adblockplusui they called the sc
hub 2018/06/17 23:06:08 But in adblockpluscore it is called eslint (which
hub 2018/06/18 12:15:56 Even libadblockplus use `eslint` as the script nam
12 "posttest": "npm run eslint" 12 "posttest": "npm run lint"
13 } 13 }
14 } 14 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld