Update security rules to follow new best practices for transferring value

[vince0656]

Name
Not thought of a name

Problem
Based on https://diligence.consensys.net/posts/2019/09/stop-using-soliditys-transfer-now/ we are now using .call.value() to send value. However, this means that Solium is throwing the following:
72:37 error Consider using 'transfer' in place of 'call.value()'. security/no-call-value

Description
We are suppressing this warning for now but would be great if we could fix

Priority
Med/Low

[vince0656]

@jamesmorgan

[duaraghav8]

Hi @vince0656 I just noticed this issue, I'm not seeing a lot of notifs on my github, its been messed up for a while. Sorry for responding so late.

To be clear, does Ethereum and/or Consensys encourage the use of call.value()?
If yes, is there any authoritative document on this? Or is this diligence document the most authoritative in your opinion?

[jamesmorgan]

@duaraghav8 I think its in the doc @vince0656 linked above https://diligence.consensys.net/posts/2019/09/stop-using-soliditys-transfer-now/