yannickcr / eslint-plugin-react / 4602

97%

Build Type

push

travis-ci

Committed by Jordan Harband

Commit Message

[New] `jsx-no-constructed-context-values`: add new rule which checks when the value passed to a Context Provider will cause needless rerenders

Adds a new rule that checks if the value prop passed to a Context Provider will cause needless rerenders. A common example is inline object
declaration such as:
```js
<Context.Provider value={{foo: 'bar'}}>
...
</Context.Provider>
```
which will cause the Context to rerender every time this is run because the object is defined with a new reference. This can lead to
performance hits since not only will this rerender all the children of a context, it will also update all consumers of the value. The
search React does to search the tree can also be very expensive. Some other instances that this rule covers are: array declarations,
function declarations (i.e. arrow functions), or new class object instantians.

Run Details

4768 of 5034 branches covered (94.72%)

73 of 73 new or added lines in 1 file covered. (100.0%)

5625 of 5770 relevant lines covered (97.49%)

12462.28 hits per line