eslint: relax no-return-assign to 'except-parens'
eslint: relax no-return-assign to 'except-parens'

a common pattern to set a ref for a react element is:
<div ref={e => this.refName = e} />

But if you are using our eslint rules (currently set to 
"no-return-assign": ["error", "always”]) it will force you to do:
<div ref={e => {this.refName = e}} />

And now, if you are using prettier, it will format that as:
<div ref={e => {
  this.refName = e
}} />

That seems needlessly verbose. With this change you can now do:
<div ref={e => (this.refName = e)} />

And both eslint and prettier will be ok with that

Test plan:
* verify functions that return an assignment do what you want in eslint

Change-Id: I56abfdb21a7bf3f5b3eae8088f27749f41f2371c
Reviewed-on: https://gerrit.instructure.com/131788
Tested-by: Jenkins
Reviewed-by: Felix Milea-Ciobanu <9384aa226ac9d3237ed0bc52e18ffcb251ee0bf7@instructure.com>
Product-Review: Ryan Shaw <ea3cd978650417470535f3a4725b6b5042a6ab59@instructure.com>
QA-Review: Ryan Shaw <ea3cd978650417470535f3a4725b6b5042a6ab59@instructure.com>

djbender/canvas-lms,instructure/canvas-lms,instructure/canvas-lms,sfu/canvas-lms,sfu/canvas-lms,djbender/canvas-lms,instructure/canvas-lms,djbender/canvas-lms,sfu/canvas-lms,sfu/canvas-lms,instructure/canvas-lms,djbender/canvas-lms,sfu/canvas-lms,instructure/canvas-lms,instructure/canvas-lms,sfu/canvas-lms,instructure/canvas-lms,sfu/canvas-lms