.eslintignore - Issue 29374674: Issue 4864 - Start using ESLint for adblockpluschrome

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments.

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
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
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
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
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Side by Side Diff: .eslintignore

Issue 29374674: Issue 4864 - Start using ESLint for adblockpluschrome (Closed)

Patch Set: Use var for ext declarations again Created Feb. 8, 2017, 9:02 a.m.

Left:
Right:

Use n/p to move between diff chunks; N/P to move between comments.

Jump to:

OLD	NEW
(Empty)
	1 lib/punycode.js
	Sebastian Noack 2017/02/09 01:04:48 I really don't like how the ESLint configuration i I really don't like how the ESLint configuration is spread out over multiple configuration files (not to mention the inline configurations). Wouldn't it be possible to files to be ignored in .eslintrc.json, at least? Also let me know if you have a better idea for qunit/.eslintrc.json. kzar 2017/02/20 10:27:28 I don't think there's another way, other than to s Show quoted text On 2017/02/09 01:04:48, Sebastian Noack wrote: > I really don't like how the ESLint configuration is spread out over multiple > configuration files (not to mention the inline configurations). Wouldn't it be > possible to files to be ignored in .eslintrc.json, at least? I don't think there's another way, other than to specify the files to ignore as command line arguments. http://eslint.org/docs/user-guide/configuring#ignoring-files-and-directories https://github.com/eslint/eslint/issues/3529 Show quoted text > Also let me know if you have a better idea for > qunit/.eslintrc.json. The only other way I see is to add the qunit env to adblockpluschrome's root .eslintrc.json which IMO is worse.
	2 lib/publicSuffixList.js

	3 jquery-ui/js/*

	4 qunit/qunit.js

OLD	NEW

« no previous file with comments | « no previous file | .eslintrc.json » ('j') | chrome/ext/background.js » ('J')