third_party/WebKit/Source/devtools/package.json - Issue 2338753003: DevTools: Make eslint mandatory and check node.js/npm modules in presubmit

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: third_party/WebKit/Source/devtools/package.json

Issue 2338753003: DevTools: Make eslint mandatory and check node.js/npm modules in presubmit (Closed)

Patch Set: Fix copyright year Created 4 years, 3 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« no previous file with comments | « third_party/WebKit/Source/devtools/PRESUBMIT.py ('k') | third_party/WebKit/Source/devtools/scripts/lint_javascript.py » ('j') | third_party/WebKit/Source/devtools/scripts/lint_javascript.py » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 {	1 {

2 "name": "chrome-devtools-frontend",	2 "name": "chrome-devtools-frontend",

3 "description": "Chrome DevTools UI",	3 "description": "Chrome DevTools UI",

4 "scripts": {	4 "scripts": {

5 "server": "node scripts/hosted_mode/server.js",	5 "server": "node scripts/hosted_mode/server.js",

6 "test": "echo \"Error: no test specified\" && exit 1",	6 "test": "echo \"Error: no test specified\" && exit 1",

7 "lint": "eslint -c front_end/.eslintrc.js --ignore-path front_end/.eslintign ore front_end"	7 "lint": "eslint -c front_end/.eslintrc.js --ignore-path front_end/.eslintign ore front_end"

8 },	8 },

9 "repository": {	9 "repository": {

10 "type": "git",	10 "type": "git",

11 "url": "git+https://github.com/ChromeDevTools/devtools-frontend.git"	11 "url": "git+https://github.com/ChromeDevTools/devtools-frontend.git"

12 },	12 },

13 "keywords": [	13 "keywords": [

14 "devtools",	14 "devtools",

15 "chrome",	15 "chrome",

16 "chromium",	16 "chromium",

17 "blink",	17 "blink",

18 "debugger"	18 "debugger"

19 ],	19 ],

20 "author": "The Chromium Authors",	20 "author": "The Chromium Authors",

21 "license": "SEE LICENSE IN https://chromium.googlesource.com/chromium/src/+/ma ster/LICENSE",	21 "license": "SEE LICENSE IN https://chromium.googlesource.com/chromium/src/+/ma ster/LICENSE",

22 "bugs": {	22 "bugs": {

23 "url": "https://bugs.chromium.org/p/chromium/issues/list?can=2&q=component:P latform%3EDevTools%20&sort=-opened&colspec=ID%20Stars%20Owner%20Summary%20Modifi ed%20Opened"	23 "url": "https://bugs.chromium.org/p/chromium/issues/list?can=2&q=component:P latform%3EDevTools%20&sort=-opened&colspec=ID%20Stars%20Owner%20Summary%20Modifi ed%20Opened"

24 },	24 },

25 "homepage": "https://devtools.chrome.com"	25 "homepage": "https://devtools.chrome.com",

	26 "devDependencies": {

	27 "eslint": "3.5.0"

	28 }

26 }	29 }

OLD	NEW