Linter complains when using arrow function for refs

[mrscobbler]

I'm using arrow function for refs as explained in the docs here: https://facebook.github.io/react/docs/more-about-refs.html

but the linter complains with this:

217:54 warning Arrow function should not return assignment

Is there an issue with the way the docs describe how to use arrow functions with refs? Should the docs be updated?

[vjeux]

You are probably writing

<TextInput ref={(c) => this._input = c} />

if you add {} around the function body it should remove the warning:

<TextInput ref={(c) => { this._input = c; }} />

[mrscobbler]

Oh yep, that was the problem. Thanks for the clarification.

[gaearon]

I’m actually not sure we should keep this rule.
Callback refs are verbose enough by themselves, and this makes them worse.

[gaearon]

@mrscobbler Would you like to send a PR to remove this rule? It should be in config/eslint.js. I don’t think it’s valuable enough to justify the confusion. If ESLint revisits eslint/eslint#5150 (comment) we can add it back with that option.

[mrscobbler]

Yes, sure I'll take care of it.

[mrscobbler]

Just submitted a pull request: #529

[gaearon]

Fixed via #529, will be out in the next version.
Thanks!