merge master into feature

.gitignore

@@ -15,6 +15,7 @@ logs
 reports
 build
 node_modules
+!tests/**/node_modules
 npm-debug.log
 sftp-config.json
 

.travis.yml

@@ -1,6 +1,7 @@
 sudo: false
 language: node_js
 node_js:
+  - '10'
   - '9'
   - '8'
   - '7'
@@ -15,36 +16,25 @@ script:
   - 'if [ -n "${PRETEST-}" ]; then npm run pretest ; fi'
   - 'if [ -n "${TEST-}" ]; then npm run unit-test ; fi'
 env:
-  - TEST=true
+  global:
+    - TEST=true
+  matrix:
+    - ESLINT=5
+    - ESLINT=4
+    - ESLINT=3
 after_success:
   - 'if [ -n "${TEST-}" ]; then npm run coveralls ; fi'
 matrix:
   fast_finish: true
   include:
     - node_js: 'lts/*'
       env: PRETEST=true
-    - node_js: '9'
-      env: TEST=true ESLINT=next
-    - node_js: '8'
-      env: TEST=true ESLINT=next
-    - node_js: '7'
-      env: TEST=true ESLINT=next
-    - node_js: '6'
-      env: TEST=true ESLINT=next
-    - node_js: '9'
-      env: TEST=true ESLINT=3
-    - node_js: '8'
-      env: TEST=true ESLINT=3
-    - node_js: '7'
-      env: TEST=true ESLINT=3
-    - node_js: '6'
-      env: TEST=true ESLINT=3
+  exclude:
     - node_js: '5'
-      env: TEST=true ESLINT=3
+      env: ESLINT=5
     - node_js: '4'
-      env: TEST=true ESLINT=3
+      env: ESLINT=5
   allow_failures:
     - node_js: '9'
     - node_js: '7'
     - node_js: '5'
-    - env: TEST=true ESLINT=next

CHANGELOG.md

@@ -3,6 +3,100 @@ All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 This change log adheres to standards from [Keep a CHANGELOG](http://keepachangelog.com).
 
+## [7.11.1] - 2018-08-14
+### Fixed
+* stop crashing when assigning to propTypes ([#1932][], @alexzherdev)
+
+### Changed
+* Fix changelog links ([#1926][], @ferhatelmas)
+* Fix changelog links ([#1929][], @alexzherdev)
+
+[#1932]: https://github.com/yannickcr/eslint-plugin-react/pull/1932
+[#1929]: https://github.com/yannickcr/eslint-plugin-react/pull/1929
+[#1926]: https://github.com/yannickcr/eslint-plugin-react/pull/1926
+
+## [7.11.0] - 2018-08-13
+### Added
+* [`jsx-one-expression-per-line`][]: add "allow" option ([#1924][], @alexzherdev)
+* [`sort-prop-types`][]: add autofix ([#1891][], @finnp)
+* [`jsx-no-bind`][]: Add ignoreDOMComponents option ([#1868][], @alexzherdev)
+* Output a warning if React version is missing in settings ([#1857][], @alexzherdev)
+
+### Fixed
+* [`destructuring-assignment`][]: Ignore class properties ([#1909][], @alexandernanberg)
+* [`destructuring-assignment`][], component detection: ignore components with confidence = 0 ([#1907][], @alexzherdev)
+* [`boolean-prop-naming`][]: Handle inline Flow type ([#1905][], @alexzherdev)
+* [`jsx-props-no-multi-spaces`][]: Handle member expressions ([#1890][], @alexzherdev)
+* [`sort-comp`][]: Allow methods to belong to any matching group ([#1858][], @nosilleg)
+* [`jsx-sort-props`][]: Fix `reservedFirst` ([#1883][], @fleischie)
+* [`prop-types`][]: (flow) Stop crashing on undefined or null properties ([#1860][], @nicholas-l)
+* [`no-unknown-property`][]: Make attribute "charset" valid ([#1863][], @silvenon)
+* [`no-deprecated`][]: report identifier AST node instead of the class node ([#1854][], @jsnajdr)
+* [`button-has-type`][]: Account for pragma ([#1851][], @alexzherdev)
+* [`button-has-type`][]: improve error message when an identifier is used as the value ([#1874][], @ljharb)
+* support JSXText nodes alongside Literal nodes (@ljharb)
+
+### Changed
+* Extract propTypes detection code ([#1911][], @alexzherdev)
+* Fix broken links in changelog ([#1849][], @alexzherdev)
+* [`no-unused-state`][]: combine spread visitors (@ljharb)
+* [`jsx-one-expression-per-line`][]: Fix JSX Syntax in docs ([#1867][], @peter-mouland)
+* [`jsx-max-depth`][], [`jsx-sort-default-props`][]: add missing docs urls ([#1880][], @flyerhzm)
+* [`jsx-indent`][]: add test cases ([#1892][], @alexzherdev)
+* [`prop-types`][]: add test cases ([#1898][], @alexzherdev)
+* Add a helper function for determining function-like expressions ([#1914][], @alexzherdev)
+* [`jsx-props-no-multi-spaces`][]: update docs ([#1918][], @BenRichter)
+
+[#1924]: https://github.com/yannickcr/eslint-plugin-react/pull/1924
+[#1918]: https://github.com/yannickcr/eslint-plugin-react/pull/1918
+[#1914]: https://github.com/yannickcr/eslint-plugin-react/pull/1914
+[#1911]: https://github.com/yannickcr/eslint-plugin-react/pull/1911
+[#1909]: https://github.com/yannickcr/eslint-plugin-react/pull/1909
+[#1907]: https://github.com/yannickcr/eslint-plugin-react/pull/1907
+[#1905]: https://github.com/yannickcr/eslint-plugin-react/pull/1905
+[#1898]: https://github.com/yannickcr/eslint-plugin-react/pull/1898
+[#1892]: https://github.com/yannickcr/eslint-plugin-react/pull/1892
+[#1891]: https://github.com/yannickcr/eslint-plugin-react/pull/1891
+[#1890]: https://github.com/yannickcr/eslint-plugin-react/pull/1890
+[#1883]: https://github.com/yannickcr/eslint-plugin-react/pull/1883
+[#1880]: https://github.com/yannickcr/eslint-plugin-react/pull/1880
+[#1874]: https://github.com/yannickcr/eslint-plugin-react/issues/1874
+[#1868]: https://github.com/yannickcr/eslint-plugin-react/pull/1868
+[#1867]: https://github.com/yannickcr/eslint-plugin-react/pull/1867
+[#1863]: https://github.com/yannickcr/eslint-plugin-react/pull/1863
+[#1860]: https://github.com/yannickcr/eslint-plugin-react/pull/1860
+[#1858]: https://github.com/yannickcr/eslint-plugin-react/pull/1858
+[#1857]: https://github.com/yannickcr/eslint-plugin-react/pull/1857
+[#1854]: https://github.com/yannickcr/eslint-plugin-react/pull/1854
+[#1851]: https://github.com/yannickcr/eslint-plugin-react/pull/1851
+[#1849]: https://github.com/yannickcr/eslint-plugin-react/pull/1849
+
+## [7.10.0] - 2018-06-24
+### Added
+* Allow eslint ^5 ([#1843][] @papandreou, @ljharb)
+* [`no-unsafe`][] rule ([#1831][], [#1830][] @sergei-startsev)
+* [`no-will-update-set-state`][]: Account for `UNSAFE_` methods ([#1845][], [#1844][] @alexzherdev)
+
+### Fixed
+* [`no-typos`][]: Fix static propTypes handling ([#1827][], [#1677][] @alexzherdev)
+* [`destructuring-assignment`][]: Allow LHS ([#1825][], [#1728][] @alexzherdev)
+* [`no-unused-prop-types`][]: Fix crash when encountering mixed union and intersection flow types ([#1806][] @yannickcr)
+
+### Changed
+* Typo fixes in [`jsx-no-target-blank`][] ([#1805][] @ferhatelmas))
+
+[#1845]: https://github.com/yannickcr/eslint-plugin-react/pull/1845
+[#1844]: https://github.com/yannickcr/eslint-plugin-react/issues/1844
+[#1843]: https://github.com/yannickcr/eslint-plugin-react/pull/1843
+[#1831]: https://github.com/yannickcr/eslint-plugin-react/pull/1831
+[#1830]: https://github.com/yannickcr/eslint-plugin-react/issues/1830
+[#1827]: https://github.com/yannickcr/eslint-plugin-react/pull/1827
+[#1825]: https://github.com/yannickcr/eslint-plugin-react/pull/1825
+[#1806]: https://github.com/yannickcr/eslint-plugin-react/issues/1806
+[#1805]: https://github.com/yannickcr/eslint-plugin-react/pull/1805
+[#1728]: https://github.com/yannickcr/eslint-plugin-react/issues/1728
+[#1677]: https://github.com/yannickcr/eslint-plugin-react/issues/1677
+
 ## [7.9.1] - 2018-06-03
 * Nothing was fixed; this is a republish with some updated deps. ([#1804][] @ljharb)
 
@@ -106,6 +200,7 @@ This change log adheres to standards from [Keep a CHANGELOG](http://keepachangel
 [#1670]: https://github.com/yannickcr/eslint-plugin-react/pull/1670
 [#1669]: https://github.com/yannickcr/eslint-plugin-react/pull/1669
 [#1666]: https://github.com/yannickcr/eslint-plugin-react/pull/1666
+[#1665]: https://github.com/yannickcr/eslint-plugin-react/pull/1665
 [#1655]: https://github.com/yannickcr/eslint-plugin-react/pull/1655
 [#1610]: https://github.com/yannickcr/eslint-plugin-react/pull/1610
 [#1414]: https://github.com/yannickcr/eslint-plugin-react/pull/1414
@@ -2235,3 +2330,4 @@ If you're still not using React 15 you can keep the old behavior by setting the
 [`jsx-sort-default-props`]: docs/rules/jsx-sort-default-props.md
 [`jsx-max-depth`]: docs/rules/jsx-max-depth.md
 [`jsx-props-no-multi-spaces`]: docs/rules/jsx-props-no-multi-spaces.md
+[`no-unsafe`]: docs/rules/no-unsafe.md

README.md

@@ -117,6 +117,7 @@ Enable the rules that you would like to use.
 * [react/no-this-in-sfc](docs/rules/no-this-in-sfc.md): Prevent using `this` in stateless functional components
 * [react/no-unescaped-entities](docs/rules/no-unescaped-entities.md): Prevent invalid characters from appearing in markup
 * [react/no-unknown-property](docs/rules/no-unknown-property.md): Prevent usage of unknown DOM property (fixable)
+* [react/no-unsafe](docs/rules/no-unsafe.md): Prevent usage of `UNSAFE_` methods
 * [react/no-unused-prop-types](docs/rules/no-unused-prop-types.md): Prevent definitions of unused prop types
 * [react/no-unused-state](docs/rules/no-unused-state.md): Prevent definitions of unused state properties
 * [react/no-will-update-set-state](docs/rules/no-will-update-set-state.md): Prevent usage of `setState` in `componentWillUpdate`
@@ -157,6 +158,7 @@ Enable the rules that you would like to use.
 * [react/jsx-no-undef](docs/rules/jsx-no-undef.md): Disallow undeclared variables in JSX
 * [react/jsx-one-expression-per-line](docs/rules/jsx-one-expression-per-line.md): Limit to one expression per line in JSX
 * [react/jsx-curly-brace-presence](docs/rules/jsx-curly-brace-presence.md): Enforce curly braces or disallow unnecessary curly braces in JSX
+* [react/jsx-fragments](docs/rules/jsx-fragments.md): Enforce shorthand or standard form for React fragments
 * [react/jsx-pascal-case](docs/rules/jsx-pascal-case.md): Enforce PascalCase for user-defined JSX components
 * [react/jsx-props-no-multi-spaces](docs/rules/jsx-props-no-multi-spaces.md): Disallow multiple spaces between inline JSX props (fixable)
 * [react/jsx-sort-default-props](docs/rules/jsx-sort-default-props.md): Enforce default props alphabetical sorting

appveyor.yml

@@ -19,6 +19,7 @@ matrix:
     - nodejs_version: "9"
     - nodejs_version: "7"
     - nodejs_version: "5"
+    - nodejs_version: "4"
 
 platform:
   - x86

docs/rules/destructuring-assignment.md

@@ -85,3 +85,24 @@ const Foo = class extends React.PureComponent {
     return <div>{this.state.title}</div>;
   }
 };
+```
+
+## Rule Options
+
+```js
+...
+"react/destructuring-assignment": [<enabled>, "always", { "ignoreClassFields": <boolean> }]
+...
+```
+
+### `ignoreClassFields`
+
+When `true` the rule will ignore class field declarations.
+
+The following patterns are then considered okay and do not cause warnings:
+
+```jsx
+class Foo extends React.PureComponent {
+  bar = this.props.bar
+}
+```

docs/rules/display-name.md

@@ -33,7 +33,7 @@ var Hello = createReactClass({
 ...
 ```
 
-### `ignoreTranspilerName`
+### `ignoreTranspilerName` (default: `false`)
 
 When `true` the rule will ignore the name set by the transpiler and require a `displayName` property in this case.
 

docs/rules/jsx-fragments.md

@@ -0,0 +1,57 @@
+# Enforce shorthand or standard form for React fragments (react/jsx-fragments)
+
+In JSX, a React fragment is created either with `<React.Fragment>...</React.Fragment>`, or, using the shorthand syntax, `<>...</>`. This rule allows you to enforce one way or the other.
+
+Support for fragments was added in React v16.2, so the rule will warn on either of these forms if an older React version is specified in [shared settings][shared_settings].
+
+## Rule Options
+
+```js
+...
+"react/jsx-fragments": [<enabled>, <mode>]
+...
+```
+
+### `syntax` mode
+
+This is the default mode. It will enforce the shorthand syntax for React fragments, with one exception. [Keys or attributes are not supported by the shorthand syntax][short_syntax], so the rule will not warn on standard-form fragments that use those.
+
+The following pattern is considered a warning:
+
+```jsx
+<React.Fragment><Foo /></React.Fragment>
+```
+
+The following patterns are **not** considered warnings:
+
+```jsx
+<><Foo /></>
+```
+
+```jsx
+<React.Fragment key="key"><Foo /></React.Fragment>
+```
+
+### `element` mode
+
+This mode enforces the standard form for React fragments.
+
+The following pattern is considered a warning:
+
+```jsx
+<><Foo /></>
+```
+
+The following patterns are **not** considered warnings:
+
+```jsx
+<React.Fragment><Foo /></React.Fragment>
+```
+
+```jsx
+<React.Fragment key="key"><Foo /></React.Fragment>
+```
+
+[fragments]: https://reactjs.org/docs/fragments.html
+[shared_settings]: /README.md#configuration
+[short_syntax]: https://reactjs.org/docs/fragments.html#short-syntax

docs/rules/jsx-no-bind.md

@@ -7,59 +7,70 @@ A `bind` call or [arrow function](https://developer.mozilla.org/en-US/docs/Web/J
 The following patterns are considered warnings:
 
 ```jsx
-<div onClick={this._handleClick.bind(this)}></div>
+<Foo onClick={this._handleClick.bind(this)}></Foo>
 ```
 ```jsx
-<div onClick={() => console.log('Hello!')}></div>
+<Foo onClick={() => console.log('Hello!')}></Foo>
 ```
 
 The following patterns are **not** considered warnings:
 ```jsx
-<div onClick={this._handleClick}></div>
+<Foo onClick={this._handleClick}></Foo>
 ```
 
 ## Rule Options
 
 ```js
 "react/jsx-no-bind": [<enabled>, {
+  "ignoreDOMComponents": <boolean> || false,
   "ignoreRefs": <boolean> || false,
   "allowArrowFunctions": <boolean> || false,
   "allowFunctions": <boolean> || false,
   "allowBind": <boolean> || false
 }]
 ```
 
+### `ignoreDOMComponents`
+
+When `true` the following are **not** considered warnings:
+
+```jsx
+<div onClick={this._handleClick.bind(this) />
+<span onClick={() => console.log("Hello!")} />
+<button onClick={function() { alert("1337") }} />
+```
+
 ### `ignoreRefs`
 
 When `true` the following are **not** considered warnings:
 
 ```jsx
-<div ref={c => this._div = c} />
-<div ref={this._refCallback.bind(this)} />
+<Foo ref={c => this._div = c} />
+<Foo ref={this._refCallback.bind(this)} />
 ```
 
 ### `allowArrowFunctions`
 
 When `true` the following is **not** considered a warning:
 
 ```jsx
-<div onClick={() => alert("1337")} />
+<Foo onClick={() => alert("1337")} />
 ```
 
 ### `allowFunctions`
 
 When `true` the following is not considered a warning:
 
 ```jsx
-<div onClick={function () { alert("1337") }} />
+<Foo onClick={function () { alert("1337") }} />
 ```
 
 ### `allowBind`
 
 When `true` the following is **not** considered a warning:
 
 ```jsx
-<div onClick={this._handleClick.bind(this)} />
+<Foo onClick={this._handleClick.bind(this)} />
 ```
 
 ## Protips