Harrison Shoff
b79e951280
[eslint-v2][arrays] update array-callback-return good/bad ex
2016-02-20 23:33:39 -08:00
Harrison Shoff
822c0dfdfe
[eslint-v2][react] jsx-sort-prop-types => sort-prop-types
2016-02-20 13:10:42 -08:00
Harrison Shoff
07948531dc
[eslint-v2][react] set ignoreTranspilerName to false
2016-02-20 13:05:49 -08:00
Harrison Shoff
133fc51e21
[eslint-v2][react] add static-methods to top of sort-comp order
2016-02-20 13:02:15 -08:00
Harrison Shoff
ff0adbe291
[eslint-v2][react] acceptTranspilerName => ignoreTranspilerName
2016-02-20 12:59:00 -08:00
Harrison Shoff
e0959d0f1d
[eslint-v2] fix rule links in readme
2016-02-14 19:04:12 -08:00
Harrison Shoff
65609373bf
[eslint-v2][template strings] add template-curly-spacing rule, fix #716
2016-02-14 18:56:49 -08:00
Harrison Shoff
1f12a12be4
[eslint-v2] fix empty constructor example
2016-02-14 18:48:31 -08:00
Harrison Shoff
e1a087fbb1
[eslint-v2] add prefer-rest-params
2016-02-14 18:45:44 -08:00
Harrison Shoff
c5b4f05879
[eslint-v2] add one-var-declaration-per-line
2016-02-14 18:39:32 -08:00
Harrison Shoff
e6cbcf4cc5
[eslint-v2][whitespace] add no-whitespace-before-property rule
2016-02-14 18:35:23 -08:00
Harrison Shoff
92df6357b3
[eslint-v2][variables] add no-self-assign
2016-02-14 18:30:28 -08:00
Harrison Shoff
3762c9add8
[eslint-v2] add no-new-symbol rule
2016-02-14 14:58:28 -08:00
Harrison Shoff
ba10353e9b
[eslint-v2][arrow functions] improve examples
2016-02-14 14:50:49 -08:00
Harrison Shoff
6a03a32915
[eslint-v2][arrow functions] add no-confusing-arrow rule
2016-02-14 14:47:23 -08:00
Harrison Shoff
172dffbb52
[eslint-v2][whitespace] add newline-per-chained-call rule
2016-02-14 11:48:04 -08:00
Harrison Shoff
fbd9c35dd3
[eslint-v2][arrays] add array-callback-return rule
2016-02-14 11:14:38 -08:00
Harrison Shoff
a126d0b85c
[eslint-v2] enforce yield-start-spacing, no-unused-labels, no-extra-label
2016-02-14 12:56:03 -08:00
Harrison Shoff
c05b2da9db
[eslint-v2] add new rules, disabled:
...
- `id-blacklist`
- `no-extra-label`
- `no-implicit-globals`
- `no-restricted-imports`
- `no-unmodified-loop-condition`
- `no-unused-labels`
- `sort-imports`
- `yield-star-spacing`
2016-02-14 02:50:51 -08:00
Harrison Shoff
91b83650b0
[eslint-v2][constructors] fix bad + good examples
2016-02-14 02:34:47 -08:00
Harrison Shoff
1cbc628c49
[eslint-v2][constructors] disallow unnecessary constructors
2016-02-13 13:57:38 -08:00
Harrison Shoff
5109c84926
[eslint-v2] fix no-labels rule
2016-02-13 13:41:52 -08:00
Harrison Shoff
75807b9d5e
[eslint-v2] no gens, no dup props in obj literal
2016-02-12 11:42:15 -08:00
Harrison Shoff
3aa8c58487
[eslint-v2] fix keyword-spacing style and sorting
2016-02-12 11:36:26 -08:00
Harrison Shoff
4f32315b24
[eslint-v2] ecmaFeatures => parserOptions
2016-02-12 11:36:08 -08:00
Harrison Shoff
dff5099238
[eslint-v2] set es6 env to true
2016-02-12 11:30:11 -08:00
Harrison Shoff
da4213c6dd
[eslint-v2] fix no-labels rule
2016-02-12 11:29:20 -08:00
Harrison Shoff
1efb11ca93
[eslint-v2] space-after/before/return/throw/case => keyword-spacing
2016-02-12 11:11:22 -08:00
Harrison Shoff
d49a1ed670
[eslint-v2] no-empty-label => no-labels
2016-02-12 11:10:30 -08:00
Harrison Shoff
1bb72ab9e2
[Deps] update eslint to v2, eslint-plugin-react to v4
2016-02-12 11:09:39 -08:00
Sergey Fursov
4574659833
rearrange comma-dangle rule to match es5/es6 codestyles, fixes #741
2016-02-21 01:31:20 +03:00
Harrison Shoff
aa9add108e
[pkg] add harry to contributors
2016-02-12 11:37:39 -08:00
Jordan Harband
8b58d8770c
v5.0.1
2016-02-13 11:11:39 -08:00
Jordan Harband
d952efed24
[Fix] eslint peerDep should not include breaking changes.
2016-02-13 11:10:48 -08:00
Jordan Harband
44cc90f31c
Merge pull request #715 from lencioni/sort-comp-statics
...
[guide] [react] Sort static methods above constructor
2016-02-04 15:26:39 -08:00
Joe Lencioni
e007ec5be9
Sort static methods above constructor for React
...
I think it makes more sense to put static methods above the constructor
in classes. I would like to update the ESLint configuration to match
this, but it looks like the react/sort-comp rule does not support it
quite yet.
https://github.com/yannickcr/eslint-plugin-react/issues/128
2016-02-04 11:01:17 -08:00
Ray Sinlao
24e7fd0c14
Added parenthesis to 'good' example in 8.2
2016-02-04 00:00:58 -08:00
Jordan Harband
06fab80dfa
v5.0.0
2016-02-03 18:29:18 -08:00
Jordan Harband
a8461e0453
[eslint config] [fix] fix npm run lint
2016-02-03 18:28:07 -08:00
Jordan Harband
6f1d92851c
[eslint config] [fix] s/no-case-declaration/no-case-declarations/g (from #712 )
2016-02-03 18:27:59 -08:00
Jordan Harband
b6920af581
[eslint config] [dev deps] update eslint-plugin-react, react, tape
2016-02-03 17:26:33 -08:00
Jordan Harband
7e15ccf4cb
[eslint config] [breaking] disallow unneeded ternary expressions.
2016-02-03 17:25:00 -08:00
Jordan Harband
fa4925be82
Merge pull request #712 from lencioni/no-case-declarations
...
[eslint config] [breaking] Avoid lexical declarations in case/default clauses
2016-02-03 13:04:51 -08:00
Joe Lencioni
4d74fe1f5a
Add example of a case clause that does not need a block
...
As @kesne and @ljharb pointed out, this section was unclear about
whether or not you should always include blocks for all case clauses. To
make things clearer, I am adding a case clause that does not need a
block to the good example.
We decided that we should only require blocks for case clauses that
actually need them because it matches the as-needed spirit of section
3.8 ("Only quote properties that are invalid identifiers"). Perhaps if
there was an as-needed but consistent setting for the ESLint rule, we
would consider revising this a little, but this seems good enough for
now.
2016-02-03 12:23:51 -08:00
Joe Lencioni
38d9b34b27
Avoid lexical declarations in case/default clauses
...
This commit adds guidance warning people to avoid lexical declarations
in case/default clauses and enables the corresponding ESLint rule.
http://eslint.org/docs/rules/no-case-declarations.html
We didn't have a section on switch statements yet, so I thought about
starting one. It seemed like it would best fit between section 15
(Comparison Operators & Equality) and section 16 (Blocks), but I didn't
want to mess up all of the following numberings, since people probably
have references to them. I considered adding this near the end to
minimize this effect, but it really seemed to belong near these other
things. I landed on appending it to Section 15 (Comparison Operators &
Equality) and I think it sorta fits there since switch statements are a
related concept.
2016-02-02 16:11:53 -08:00
Jordan Harband
206e13b185
[guide] remove trailing whitespace
2016-02-02 10:33:18 -08:00
Barry Gitarts
d293e74872
add section 15.5 - Ternaries
...
add link to the relevant eslint rule
2016-02-02 13:11:55 -05:00
Prayag Verma
7eb7b78513
Minor typo
...
Remove extra dot
2016-01-30 10:42:33 +05:30
Mark Sanghoon Kim
51510a43c8
Fixed capitalization and comment spacing
...
Fixed line 1219 to have a lowercase 'a' to be consistent with the style guide.
Fixed line 1831 to only have one space between the // and the =>
2016-01-27 23:14:53 -08:00
Fredrik Olovsson
784d029980
Edit sample code in 24.1 to follow rule 8.1
...
Replaced function expression with arrow function notation.
2016-01-27 11:53:57 +01:00
