Commit Graph

689 Commits

Author SHA1 Message Date
Billy Janitsch
bbcb1371e5 Remove deprecated react/jsx-quotes 2015-12-15 15:59:40 -05:00
Jordan Harband
60b01bdfb9 [eslint config] v2.1.0 eslint-config-airbnb-v2.1.0 2015-12-15 01:22:30 -08:00
Jordan Harband
0fc31eaf1c [eslint config] fix b1500397e1 2015-12-15 01:19:32 -08:00
Jordan Harband
4ab41ede1b [eslint config] [Dev Deps] update eslint 2015-12-14 23:55:21 -08:00
Jake Teton-Landis
a7541c9b22 Merge pull request #582 from jkrems/jk-require-resolve
Use require.resolve to allow nested extend
2015-12-14 23:49:00 -08:00
Jordan Harband
b1500397e1 [eslint config] enable object-shorthand rule.
Fixes #621.
2015-12-14 22:21:52 -08:00
Jordan Harband
28a83da1d1 Merge pull request #620 from OutBoxSoft/master
[Docs] Add OutBoxSoft to the list of organizations
2015-12-14 16:21:02 -08:00
Gerardo Renovales
2ab6253f85 Update README.md
Add OutBoxSoft to the list of organizations
2015-12-14 15:06:57 -05:00
Jordan Harband
5989125d2e Merge pull request #618 from kesne/jgens/react-defaults
[eslint config] [react] [minor] Including missing defaults to the react eslint; enable react/jsx-quotes
2015-12-11 23:17:25 -08:00
Jordan Gensler
d02a506377 Adding links to react rules for quick reference. 2015-12-11 23:13:30 -08:00
Jordan Gensler
e9b1ef92bf Adding links to lint rules in react styleguide. 2015-12-11 23:00:10 -08:00
Jordan Gensler
d5a636f298 [react] Including missing defaults to the react eslint 2015-12-11 18:47:48 -08:00
Jordan Harband
8c76e4f486 Merge pull request #617 from frenzzy/patch-1
[eslint config] [breaking] Require space before/after arrow function's arrow (`arrow-spacing`)
2015-12-11 00:55:22 -08:00
Vladimir Kutepov
514bd87d1a Require space before/after arrow function's arrow (arrow-spacing)
Enable [arrow-spacing](http://eslint.org/docs/rules/arrow-spacing.html) rule, code with space before/after arrow function's arrow is easier to read.
```js
() => {};
(a) => {};
a => a;
() => {'\n'};
()=> {};     /*error Missing space before =>*/
() =>{};     /*error Missing space after =>*/
(a)=> {};    /*error Missing space before =>*/
(a) =>{};    /*error Missing space after =>*/
a =>a;       /*error Missing space after =>*/
a=> a;       /*error Missing space before =>*/
()=> {'\n'}; /*error Missing space before =>*/
() =>{'\n'}; /*error Missing space after =>*/
```
2015-12-11 13:50:46 +05:00
Jordan Harband
802725d366 Merge pull request #615 from winterbe/fix15
Fix syntax highlighting in section 15
2015-12-09 01:58:14 -08:00
Benjamin Winterberg
721b8cd01f Fix syntax highlighting in section 15 2015-12-09 10:51:49 +01:00
Josh Perez
6c89f9587f Merge pull request #608 from airbnb/document-eslint-rules
Documents corresponding eslint rules
2015-12-03 15:47:52 -08:00
Josh Perez
a1efff9f6d Documents corresponding eslint rules 2015-12-03 15:39:49 -08:00
Josh Perez
f0fb395714 Merge pull request #607 from JonAbrams/patch-1
Fix example for section 7.11
2015-12-03 14:41:41 -08:00
Jon Abrams
a31c72a1e5 Fix example for section 7.11
The missing newline caused the markdown parser to misinterpret the beginning and end of the code block.
2015-12-03 14:37:52 -08:00
Jordan Harband
4c0a70e435 [eslint config] v2.0.0
- separate changelog
eslint-config-airbnb-v2.0.0
2015-12-03 13:55:09 -08:00
Jordan Harband
326100a7b9 Merge pull request #605 from airbnb/ljharb/fix_function_spacing
[breaking] Require function spacing: `function <optional name>(`
2015-12-03 13:33:45 -08:00
Jordan Harband
2c3b75d256 Merge pull request #606 from airbnb/ljharb/fix_switch_indent
[breaking] Fix switch statement indentation rule
2015-12-03 13:16:51 -08:00
Jordan Harband
4ccd04b69a I abhor switch statements, but their indentation should be:
```js
switch (foo) {
  case 'bar':
    break;
}
```

not what the plugin currently requires:
```js
switch (foo) {
case: 'bar':
  break;
}
```
2015-12-03 00:09:12 -08:00
Jordan Harband
599ca4f123 Require a space between function and (, and function and the function's name, and disallow spaces between the function's name and (. 2015-12-02 22:58:11 -08:00
Jordan Harband
6e6a2b91e1 Merge pull request #604 from Bisk/master
Add Bisk to the list of organizations
2015-12-02 20:58:08 -08:00
Josh Perez
eee4371299 Merge pull request #600 from zwhitchcox/patch-1
Add facebook's react to "In the Wild"
2015-12-02 13:46:57 -08:00
Gerardo Renovales
e754dcc7cf Update README.md
Adding Bisk to the list of organizations
2015-12-02 16:40:41 -05:00
Jordan Harband
b640c952c1 Merge pull request #602 from kesne/jeopardy
Adding jeopardybot to styleguide users.
2015-12-02 12:26:57 -08:00
Jordan Gensler
2944b9b47b Adding jeopardybot to styleguide users. 2015-12-02 12:24:01 -08:00
zwhitchcox
287dee40e5 Add facebook's react to "In the Wild"
I can't believe you guys didn't have that one! That's probably your best one.
2015-12-02 12:48:46 -05:00
Jordan Harband
33e23fb121 Merge pull request #594 from pigoz/parens-brackets-curlys
Make the spaces within parenthesis/brackes/curlys rules explicit
2015-11-30 15:18:50 -08:00
Stefano Pigozzi
6debbcdd73 Add Section 18.11 - Add spaces inside curly braces
Fixes #593
2015-11-28 18:47:46 +01:00
Stefano Pigozzi
ee3759a7ed Add Section 18.10 - No spaces inside brackets
See #593
2015-11-28 13:29:47 +01:00
Stefano Pigozzi
2557a5bca6 Add Section 18.9 - No spaces inside parentheses
See #593
2015-11-28 13:26:37 +01:00
Jake Teton-Landis
e0326137d4 [eslint config] v1.0.2 eslint-config-airbnb-v1.0.2 2015-11-26 15:30:30 -08:00
Jordan Harband
7476c31246 Merge pull request #592 from ariporad/patch-1
[rules: es6] enable restParams
2015-11-26 00:55:51 -06:00
Ari Porad
e424a249e3 Enable rest params! 2015-11-25 20:48:48 -08:00
Josh Perez
639e3c709a Merge pull request #584 from jonathanp/master
Avoid the word "mutate" while explaining `const`
2015-11-17 11:58:17 -08:00
jpersson
a3495e07f4 Avoid the word "mutate" while explaining const 2015-11-17 13:12:02 -05:00
Jan Krems
c501e0d5a6 Use require.resolve to allow nested extend 2015-11-16 13:40:38 -08:00
Jake Teton-Landis
60480c05ea Merge pull request #578 from adrienverge/enforce-rule-18.5
[eslint] Enforce rule 18.5
2015-11-11 10:57:24 -08:00
Adrien Vergé
09c8589e44 [eslint] Enforce rule 18.5
Rule 18.5 [1] states that files shall end with a single newline
character. Until now, this was not checked because there was no such
option in eslint.

Since version 1.8.0, eslint provides the ability to do that. See pull
request on eslint [2] for details on implementation and usage. Let's use
it in airbnb eslint plugin now!

[1]: https://github.com/airbnb/javascript#18.5
[2]: https://github.com/eslint/eslint/pull/4266
2015-11-11 10:27:04 -08:00
Jordan Harband
87c9507bb5 Merge pull request #572 from joscha/patch-1
docs: babel-eslint not neede since 1.0.0 any more
2015-11-08 21:36:10 -08:00
Joscha Feth
aa6ca6ff68 docs: babel-eslint not neede since 1.0.0 any more 2015-11-09 14:44:17 +11:00
Jordan Harband
d9b5585be4 [eslint config] v1.0.0 eslint-config-airbnb-v1.0.0 2015-11-08 11:12:17 -08:00
Jordan Harband
c0ef4540e5 Merge pull request #570 from airbnb/ljharb/update_eslint
[eslint config] update `eslint` to require 1.0 or greater
2015-11-06 16:36:21 -08:00
Jordan Harband
d432f58655 Add eslint as a peer dep, 1.0 or greater 2015-11-06 16:32:28 -08:00
Jordan Harband
0b7c979d82 [eslint] Remove babel-eslint dependency. 2015-11-06 15:16:49 -08:00
Jordan Harband
88603b8445 [eslint] update eslint, eslint-plugin-react, react, tape. 2015-11-06 13:09:34 -08:00