Jordan Harband
3b18fea1ed
[Tests] only run tests in non-lint per-package travis job
2019-08-09 14:20:40 -07:00
Jordan Harband
945c62f229
[Tests] on node v12
2019-08-09 14:11:27 -07:00
Jordan Harband
820ccf7652
[eslint config] [*] [tests] use eclint instead of editorconfig-tools
2019-08-09 14:06:58 -07:00
Jordan Harband
792bb567e5
[eslint config] [*] update eslint-find-rules, eslint-plugin-import
2019-08-09 14:04:36 -07:00
Yohan Siguret
6c0a884179
[guide] fix dead jsperf links
2019-08-03 01:04:22 +02:00
Mathieu Dutto
030e23b13e
[guide] Fix npm package has usage example
2019-07-24 15:36:48 +02:00
Perttu Ristimella
036612ec3d
[inthewile] Add Happeo to users list
2019-07-04 14:59:47 +03:00
Jordan Harband
6a50516df7
[eslint config] [meta] add disabled config for new react and a11y rules
2019-07-01 17:07:38 -07:00
Jordan Harband
21493ad7e9
[eslint config] [base] [deps] update confusing-browser-globals, eslint-plugin-import, tape, babel-preset-airbnb
2019-07-01 16:26:52 -07:00
Jordan Harband
873f302b00
[eslint config] v17.1.1
2019-07-01 16:17:49 -07:00
Jordan Harband
7ea44cfc79
[eslint config] [deps] update eslint-config-airbnb-base, eslint-plugin-import, eslint-plugin-react, eslint-plugin-jsx-a11y, babel-preset-airbnb, tape
2019-07-01 16:12:08 -07:00
Jordan Harband
9fbe07f282
[eslint-config] [base] v13.2.0
2019-07-01 15:05:02 -07:00
Glenn Goossens
a9a83d9150
[guide] add link to new version of Google Style Guide
...
Add link to a newer version of this guide that includes ECMAScript 6th Edition features
2019-06-27 11:37:32 +02:00
Christian Oliff
76ff7c7f79
[editorial] [docs] correct JavaScript capitalization
2019-06-24 22:45:35 +09:00
Kevin Yue
4539dbcf17
[guide] remove redundant spaces
2019-05-24 16:29:24 +08:00
Kimball Bighorse
19701d4a0c
[inthewild] Add Coeur d'Alene Tribe
2019-05-09 09:40:43 -07:00
Joseph Nields
1310ade9d0
[eslint config] [base] [patch] Ignore property modifications of staticContext params
...
React Router passes a `staticContext` property to children to allow
signalling status codes and redirected URLs, by allowing children to
assign property values to it during render
2019-04-27 14:00:50 -07:00
Jordan Harband
5cd011dfa1
[eslint config] [base] [deps] update eslint-plugin-import
2019-04-27 23:51:44 -07:00
Jordan Harband
117d4fce49
[Tests] fix issue with eslint dev deps
2019-04-27 23:27:07 -07:00
Kévin Berthommier
447466681e
[guide] remove extra blank line
2019-04-10 10:22:57 +02:00
Anton Honcharuk
60b96d3222
[guide] Add a 19.9 bullet in a Whitespace section
...
- with explanations about preventing two blank lines from appearing consecutively in JavaScript code
2019-02-27 20:25:32 +02:00
Jordan Harband
b85baeafed
[eslint config] [base] [meta] add disabled prefer-named-capture-group rule
2019-03-03 23:23:14 -08:00
Anton Honcharuk
be07f7a020
[guide] [react] add more context about arrow functions, as props and in class fields
2019-02-24 13:28:34 +02:00
Matthias Pries
495a62aaa9
[guide] fix example 4.7 for arrays method callbacks
2019-02-13 13:52:39 +01:00
Jordan Harband
da0cf08831
[Tests] on node v11
2019-02-17 14:21:15 -08:00
Jordan Harband
a71c2d1ad3
[eslint config] add disabled jsx-fragments rule
2019-02-17 14:20:53 -08:00
Jordan Harband
834d717c7b
[eslint config] [*] [deps] update eslint-plugin-import, eslint-plugin-jsx-a11y, tape
2019-02-17 14:16:48 -08:00
Jamie
8694a87378
[eslint config] [react] [patch] Turn off react/no-multi-comp
...
Splitting up a component into multiple components can be useful for many reasons, including making important performance optimizations.
This lint rule discourages splitting up components because it forces you to: create a new file, copy over all the relevant imports, move over any local functions/values (or move them into yet another shared module), export the component (which makes your split off component now a public export), and re-import it into the new module.
Having multiple components per file shouldn't be much different from having multiple functions per file. And you shouldn't be forced to make a component a public export if it's really just an internal implementation detail of another component.
There's an argument to be made about _exporting_ multiple components, but mostly just in the context of React.lazy because it can only import default exports and you don't want something else in that file being statically imported (because that will prevent it from being code-split).
2019-02-14 11:57:20 -08:00
Pier-Luc Brault
4f15da6146
[eslint config] [*] [readme] Improve eslint config setup instructions for yarn
2019-02-07 11:48:32 -05:00
Zeeshan Ahmad
6ef45d4bab
[guide] make "good" example match "bad" example more closely
2019-02-06 14:09:18 +04:00
tclindner
96f95fa35e
[eslint config] [base] [patch] no-extraneous-dependencies: Add jest.setup.js to devDeps
2019-01-29 21:58:23 -06:00
Sharmila Jesupaul
ebe96824e2
Merge branch 'master' into translation-proposal
2019-02-05 11:51:09 -08:00
Jordan Harband
d78af39118
Merge pull request #1996 from remcohaszing/redux-devtools-extension-compose
...
[eslint config] [base] [patch] Allow use of `__REDUX_DEVTOOLS_EXTENSION_COMPOSE__`
2019-01-27 21:21:48 -08:00
Jordan Harband
d8256fc886
[eslint config] [*] [deps] update object.entries, eslint-plugin-import, eslint-plugin-jsx-a11y, eslint-plugin-react
2019-01-27 13:41:14 -08:00
Jordan Harband
25f11fb31e
[eslint config] [patch] extend no-underscore-dangle to allow for redux dev tools in the main config instead
2019-01-26 10:12:49 -08:00
Telmen Munkhdalai
1439ebe78d
[editorial] [react] fix typo
2019-01-26 16:55:34 +08:00
Remco Haszing
b6c56cb070
[eslint config] [base] [patch] Allow use of __REDUX_DEVTOOLS_EXTENSION_COMPOSE__
...
This style guide disallows the use of underscores in properties. However, the
Redux browser extension requires the use of a variable named
`__REDUX_DEVTOOLS_EXTENSION_COMPOSE__`. Since Redux is so popular with React,
it makes sense to allow this.
2019-01-21 13:39:53 +01:00
Hoang
955f4e93b8
Vietnamese translation
2019-01-13 18:55:49 +07:00
Rahil Vora
95286eb496
[guide] [react] add note about forbid-prop-types rule
2019-01-07 10:24:16 -06:00
Jordan Harband
070e6200bb
[Dev Deps] update markdownlint-cli
2018-12-28 14:32:02 -08:00
Jordan Harband
387755e8fe
[meta] add disabled no-useless-catch config
2018-12-28 14:31:03 -08:00
Jordan Harband
218c4fefbb
[eslint config] [base] [fix] disable no-var in legacy entry point
...
Fixes #1935 .
2018-12-28 13:36:01 -08:00
Emily Rohrbough
80920a07c5
Add Terra to README list
2018-12-19 13:06:08 -06:00
Honza Javorek
5d25a2ee61
capitalize means first letter uppercase, the rest lowercase
...
The examples and the reasoning indicates the author of the rule meant uppercase instead of capitalization. According to Wikipedia, capitalization is "writing a word with its first letter as a capital letter (uppercase letter) and the remaining letters in lower case", while the rule apparently tries to prevent exactly that.
2018-12-18 16:11:26 +01:00
Michael Käfer
064e3bf8d5
:) conflicting with rule 8.2 :D
2018-11-27 20:51:21 +01:00
Kevin Weber
ffe4fe967c
[guide] Clarify wording
2018-08-25 22:12:12 -07:00
ernestodebesto
69e34378c6
Examples more consistent with the guidline
...
comparison operator <= shows better the confusion when using arrow function, than just operator <
2018-09-14 17:20:22 +02:00
NullDev (Chris)
6ece1f58e9
Added NullDev
...
because why not
2018-11-21 10:51:20 +01:00
NullDev (Chris)
0d6d854b5c
[17.2] Added second anchor https://github.com/airbnb/javascript/pull/1960
...
See requested changes here: https://github.com/airbnb/javascript/pull/1960#pullrequestreview-176797787
2018-11-21 10:48:35 +01:00
Ed Morley
21b65e943c
[eslint config] [base] [deps] Switch to confusing-browser-globals
...
The `eslint-restricted-globals` package author created an equivalent
package `confusing-browser-globals` that now resides within the
create-react-app monorepo. In sidoshi/eslint-restricted-globals#2
@sidoshi suggested that we change eslint-config-airbnb-base to depend
on the CRA package instead, in an effort to consolidate the two.
Currently the two global lists are identical, so this does not change
the generated eslint config:
https://unpkg.com/eslint-restricted-globals@0.2.0/index.js
https://unpkg.com/confusing-browser-globals@1.0.5/index.js
2018-11-20 09:40:02 +00:00
