github/javascript
1
1
Fork 0
mirror of https://github.com/airbnb/javascript.git synced 2026-01-14 15:37:55 -05:00
Code Issues Packages Projects Releases Wiki Activity
686 Commits 5 Branches 102 Tags
4ab41ede1bb800fc50defa357a889d3a9aa0667a
Commit Graph

686 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
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
Jake Teton-Landis
f4e5967a0a Merge pull request #547 from chrisngobanh/padded-blocks 
Prohibit Padded Blocks and Allow Comments in the First Line of a Block
 2015-11-05 16:45:47 -08:00
Jake Teton-Landis
7b0ff1246e eslint-config-airbnb 0.1.1 eslint-config-airbnb-v0.1.1 2015-11-05 15:36:01 -08:00
Jake Teton-Landis
c5d3a3434c Merge pull request #569 from airbnb/jake/remove-id-length-rule 
[eslint] remove id length rule (too much nutrage)
 2015-11-05 15:35:20 -08:00